Ma77Ball opened a new issue, #5245: URL: https://github.com/apache/texera/issues/5245
### What happened? The pyamber test `TestCleanupFailedUpload::test_delete_object_failure_is_swallowed` fails on the Python 3.12.13 CI runner. It passes on 3.12.12 and 3.13. The test expects `delete_object` to be called once after a simulated upload failure, but on 3.12.13, it's called twice: AssertionError: Expected 'delete_object' to be called once. Called 2 times. PR #4707 fixed the same-shape bug for 3.13 by relying on `IOBase.__del__` short-circuiting on `self.closed`. On 3.12.13, the finalizer re-enters `close()` anyway, so `_cleanup_failed_upload` runs a second time. This blocks CI on every PR where Python 3.12.13 ### How to reproduce? Push any branch and let CI run, or locally with Python 3.12.13: https://github.com/apache/texera/actions/runs/26481776334/job/77980417021 ### Branch main ### Commit Hash (Optional) 8a7366f3b ### What browsers are you seeing the problem on? _No response_ ### Relevant log output ```shell FAILED test_large_binary_output_stream.py::TestCleanupFailedUpload::test_delete_object_failure_is_swallowed AssertionError: Expected 'delete_object' to be called once. Called 2 times. Calls: [call(Bucket='test-bucket', Key='path/to/object'), call(Bucket='test-bucket', Key='path/to/object')]. Runner: /opt/hostedtoolcache/Python/3.12.13/x64 ``` -- 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]
