The GitHub Actions job "Comment commands" on texera.git/main has succeeded. Run started by GitHub user tanishqgandhi1908 (triggered by tanishqgandhi1908).
Head commit for run: c57e6b054e73e3f6aec259689b526e234f86e53b / Xinyuan Lin <[email protected]> refactor(pyamber): drop dead AtomicInteger value setter (#9189) ### What changes were proposed in this PR? Removes the unused `@value.setter` on `AtomicInteger` in `amber/src/main/python/core/util/atomic.py:48-52`. Callers across the engine mutate the atomic value strictly via atomic methods (`inc`, `dec`, `get_and_inc`, `get_and_dec`, `get_and_set`) and read via the property getter (`.value`), leaving the setter unused. The corresponding unit test in `test_atomic.py` is updated to verify `value` is a read-only property (`AttributeError` on assignment). **+3/-11 lines.** Pure deletion of dead setter and regression test update. ### Any related issues, documentation, discussions? Closes #9188. ### How was this PR tested? - `python -m py_compile amber/src/main/python/core/util/atomic.py` - `python -m pytest amber/src/test/python/core/util/test_atomic.py` (11 passed) - Verified 0 callers repo-wide: `git grep -nw "AtomicInteger" | grep -v test` shows only `.inc()`, `.dec()`, `.get_and_inc()`, `.get_and_dec()`, `.get_and_set()`, `.value` (getter). ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Antigravity (Claude Opus 5.5) Report URL: https://github.com/apache/texera/actions/runs/40603222788 With regards, GitHub Actions via GitBox
