carloea2 opened a new pull request, #8234: URL: https://github.com/apache/texera/pull/8234
### What is this PR for? Route per-element batch failures through the normal exception reporter, then check the recorded exception after the batch finishes iterating. This preserves the existing attempt-all-rows behavior while preventing a workflow from silently succeeding with missing rows. Closes #8233 ### What type of PR is it? Bug Fix ### How was this PR tested? ```text python -c "import sys,pytest; sys.path[:0]=[worktree amber source, main amber source]; raise SystemExit(pytest.main(['amber/src/test/python/core/runnables/test_main_loop.py::TestMainLoop::test_a_failing_element_does_not_abandon_the_rest_of_the_batch','amber/src/test/python/core/runnables/test_data_processor.py','amber/src/test/python/core/architecture/managers/test_exception_manager.py','-q','-p','no:cacheprovider']))" 12 passed ruff check amber/src/main/python amber/src/test/python All checks passed ruff format --check amber/src/main/python amber/src/test/python 213 files already formatted ``` The regression processes two failing rows, confirms both were attempted, and confirms both failures were retained in exception history before the worker enters exception handling. Generated-by: Codex -- 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]
