C-Loftus commented on PR #969: URL: https://github.com/apache/arrow-go/pull/969#issuecomment-5049865763
Ok believe I have addressed feedback. Thanks for the review! - I will defer to you on the interface naming. https://github.com/apache/arrow-go/pull/969#discussion_r3632075763 in light of the currently exported pre-existing public interface methods. - I have changed the registration type lookup to be `FindRegisteredExtensionType` which is both lexicographically deterministic and returns one item. If this is overkill I can revert or change as you prefer. -- 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]
