robertbuessow opened a new pull request, #595:
URL: https://github.com/apache/arrow-julia/pull/595

   ## Summary
   
   - Removes the `UNRELEASED_HANDLE_COUNT` atomic counter and 
`_warn_unreleased` finalizer from `CDataHandle`
   - Removes the corresponding leak-counter tests
   - Simplifies the `CDataHandle` constructor to a one-liner
   
   The finalizer was unsafe: Julia forbids task switches in GC finalizers, and 
the `jl_safe_printf` call could still cause issues if Julia accesses array 
elements during exception handling or printing. Since `release_c_data` is 
already called explicitly in all code paths, the finalizer provided no 
meaningful safety net.
   
   ## Test plan
   
   - [ ] All existing C Data Interface tests pass
   - [ ] No GC finalizer output during test runs
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]

Reply via email to