med9110 commented on issue #6984: URL: https://github.com/apache/datafusion-comet/issues/6984#issuecomment-6214181615
Hi @andygrove, I'd be interested in contributing to #6984. I propose starting with the string and binary list hashing paths, reusing `hash_list_primitive!` to remove the duplicated null-aware loops. I'll validate that Murmur3 and xxHash64 results remain unchanged and leave the map-entry refactoring for a separate PR. The issue still has `requires-triage`. Would this focused first step be appropriate? -- 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]
