Fix leaks

[Alex-PLACET]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 86.23853% with 15 lines in your changes missing coverage. Please review.

Project coverage is 89.52%. Comparing base (7c3a27f) to head (a199f07).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
src/arrow_array_schema_proxy.cpp	65.21%	8 Missing ⚠️
src/arrow_interface/private_data_ownership.cpp	73.91%	6 Missing ⚠️
...ow_interface/arrow_array_schema_common_release.hpp	96.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #363      +/-   ##
==========================================
- Coverage   89.69%   89.52%   -0.17%     
==========================================
  Files          88       91       +3     
  Lines        6637     6693      +56     
==========================================
+ Hits         5953     5992      +39     
- Misses        684      701      +17     

Flag	Coverage Δ
unittests	89.52% <86.23%> (-0.17%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[JohanMabille]

The overall implementation looks good to me. I think it would be worth:

• gathering the children_ownership / dictionary_ownership and arrow_array_schema_common_release in a single file to avoid introducing too many files (private_data_ownership.hpp?)
• move as much implementation as possible to cpp files to reduce ABI backward incompatible changes in the future.