benbellick opened a new pull request, #1500:
URL: https://github.com/apache/arrow-go/pull/1500

   [substrait-go#161](https://github.com/substrait-io/substrait-go/issues/161) 
proposes hiding anchors and references from library users outside protobuf 
encoding and decoding. To prepare for that change, this PR reduces Arrow's 
public API exposure by removing `ExtensionIDSet` and keeping the existing 
anchor operations on a private concrete type.
   
   This is a breaking change, and I don't want to suggest it is the only path 
forward. The [discussion on Kadin's upgrade 
PR](https://github.com/apache/arrow-go/pull/1399#issuecomment-6635200067) 
raises concerns about Arrow's public API depending on substrait-go's API. This 
proposal is one step toward reducing that coupling.
   
   Anchor handling remains inside Arrow's implementation for now. This change 
narrows the public surface without changing the existing lookup behavior.
   
   ---
   Note: This PR was developed with AI assistance. All changes have been 
reviewed, and I take full responsibility for this contribution.


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