Yicong-Huang opened a new pull request, #4504: URL: https://github.com/apache/texera/pull/4504
### What changes were proposed in this PR? Delete `amber/src/test/scala/.../breakpoint/ExceptionBreakpointSpec.scala`. All 7 test cases in this file have been commented out since #941 (2020-12-21) — over 5 years of dead code. Subsequent commits only renamed packages (apache migration, pekko switch). The spec targeted the **old tuple-level breakpoint mechanism** in the Scala Amber engine (faulted/conditional/count breakpoints handled via `ReportGlobalBreakpointTriggered` / `SkipTuple` / `ModifyTuple`). The corresponding production classes (`LocalBreakpoint`, `FaultedTuple`, `SkipTuple` (the engine class, not `SkipTupleRequest`), `ModifyTuple`, `ExceptionBreakpoint`, `ReportGlobalBreakpointTriggered`) have already been removed from the codebase — `grep` against `amber/src/main` and `common/` confirms zero remaining references. This is unrelated to the modern Python UDF debugger (Udon), which is covered by frontend specs. ### Any related issues, documentation, discussions? Closes #4503 ### How was this PR tested? Pure deletion of a test file with no active test cases. No production code is touched, no behavior changes. Verified via grep that none of the surrounding (already-removed) classes are referenced anywhere. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.7) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
