yuboxx commented on PR #4222: URL: https://github.com/apache/datafusion-comet/pull/4222#issuecomment-4384924581
> Could you tell me more about the motivation for this PR? Thanks for the pointer! I was looking for some issues to work with as I'm onboarding to the codebase and stumbled on issue #477. It raised a point about dictionary type support in unhex, but apparently dictionary array is already being unpacked in advance. I'm going to remove my change in unhex.rs and only keep the unit test change to show dictionary is supported. Let me know if this makes sense. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
