The GitHub Actions job "Check binary dependency licenses" on texera.git/state-handshake-redesign has succeeded. Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).
Head commit for run: d890004c11b548a2bb4e08de3a09f1fc68991e63 / Xinyuan Lin <[email protected]> fix(amber): address review on state-handshake redesign DataProcessor.run: - Replace the post-init `_switch_context()` with a direct `_post_switch_context_checks()` call. This preserves the previous guarantee that any debug command queued during worker setup fires before the first task is processed, but does not consume a round trip on MainLoop's first switch (which would silently drop the first state/tuple/marker, see #4559). - Add an explicit `queued != 1` runtime check at the top of each iteration. MainLoop is single-threaded and sets exactly one of current_internal_marker / current_input_state / current_input_tuple before switching, so the loop body should never see zero or more than one. The previous `else: raise RuntimeError("No marker or tuple to process.")` only guarded the zero case and didn't mention state. test_main_loop.py: - Move the `_StateProcessingExecutor` test helper to module scope and expose it via a `state_processing_executor` fixture, instead of redefining the class inline in each test. The two tests previously carried near-identical copies of it. - Drop the timeouts on `test_main_loop_thread_can_process_state` and `test_main_loop_thread_can_process_state_after_tuple` from 5s to 2s -- both run in the millisecond range; 5s was just slack. Report URL: https://github.com/apache/texera/actions/runs/25148584348 With regards, GitHub Actions via GitBox
