albertlockett commented on PR #7611: URL: https://github.com/apache/arrow-rs/pull/7611#issuecomment-2970237688
> Thank you @albertlockett -- this PR looks really nice to me. It is well documented and well tested and the need makes a lot of sense > > I did have a question on potentially adding an API to determine when this code needs to be called. > > I do think adding something similar to the other dictionary builders would be great. Perhaps we can file a follow on ticket to track (with a list of the other builder types) @alamb I think this is now ready to merge as is. I created a followup issue to add similar methods to the additional dictionary builders. Feel free to assign this to me, and I'll work on it either today, or early next week. https://github.com/apache/arrow-rs/issues/7654 -- 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]
