VuMartin opened a new pull request, #8126: URL: https://github.com/apache/texera/pull/8126
### What changes were proposed in this PR? Fixes LoopStart state variables being silently overwritten by upstream state. LoopStart now detects collisions between existing loop variables and incoming state variables and raises an error instead of overwriting the loop variable. Before: <img width="913" height="629" alt="Screenshot 2026-08-27 at 6 39 29 PM" src="https://github.com/user-attachments/assets/6278d045-638d-49a4-8f66-c05be740d556"; /> <img width="772" height="247" alt="Screenshot 2026-08-28 at 12 14 43 AM" src="https://github.com/user-attachments/assets/525aa90c-c864-45e6-9e84-049c7a58cc5c"; /> After: <img width="923" height="469" alt="Screenshot 2026-08-28 at 12 03 05 AM" src="https://github.com/user-attachments/assets/1e9db3d1-8aac-45b5-9bbe-4c1b55e97cd4"; /> <img width="1085" height="498" alt="Screenshot 2026-08-28 at 12 07 52 AM" src="https://github.com/user-attachments/assets/c1ab5670-b6fe-4f8c-aebb-21318c57128e"; /> ### Any related issues, documentation, discussions? Fixes #7248 ### How was this PR tested? Added a unit test verifying that LoopStart rejects incoming state that attempts to overwrite an existing loop state variable. Ran the targeted pytest test successfully: 1 passed. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: ChatGPT (5.5 mini) -- 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]
