The GitHub Actions job "Benchmarks PR Comment" on texera.git/main has failed. Run started by GitHub user Ma77Ball (triggered by Ma77Ball).
Head commit for run: 2f9573be88e7f73e63e7f2f441187a5977fdb0a3 / Matthew B. <[email protected]> refactor(workflow-core): make JSONToMap iterative to avoid stack overflow (#5322) ### What changes were proposed in this PR? - Rewrote `JSONUtils.JSONToMap` from per-level recursion into an iterative traversal over a `(JsonNode, parentName)` worklist (a `mutable.Stack`), so nesting depth lives on the heap instead of the JVM call stack. - Per-node logic is unchanged: objects emit value entries and (in flatten mode) push their object/array children, arrays push each element keyed by 1-based index, and value nodes with a parent emit their text. Output keys and values are identical to before. - Added a `JSONUtilsSpec` case that flattens a 20,000-deep object (built programmatically to bypass Jackson's own parser nesting cap), which would previously have thrown `StackOverflowError`. ### Any related issues, documentation, or discussions? Closes: #5321 ### How was this PR tested? - Run `sbt "WorkflowCore/testOnly *JSONUtilsSpec"`; all cases pass, including the new 20,000-deep flatten case that throws `StackOverflowError` on `main`. ### Was this PR authored or co-authored using generative AI tooling? Co-authored with Claude Opus 4.7 in compliance with ASF --------- Signed-off-by: Matthew B. <[email protected]> Co-authored-by: Kunwoo (Chris) <[email protected]> Report URL: https://github.com/apache/texera/actions/runs/27515661863 With regards, GitHub Actions via GitBox
