ankitsultana opened a new pull request, #12538: URL: https://github.com/apache/pinot/pull/12538
This can help reduce memory usage for scenarios where all columns of the Primary Key are type-4 UUIDs. This can also handle scenarios where some values are not valid type-4 UUIDs, by simply switching to being a no-op hash-function. Using Murmur's 128 bit hash-function will either give the same or higher space savings, but it risks hash collision. Though chances of that are extremely low, we usually avoid any exposure for critical use-cases. **Test Plan:** Added UTs. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
