[GitHub] [pinot] richardstartin commented on pull request #7797: use `LZ4_LENGTH_PREFIXED` as default compression mode

[GitBox <gi...@apache.org>]

richardstartin commented on pull request #7797:
URL: https://github.com/apache/pinot/pull/7797#issuecomment-973411524


   > I have some concern on changing the default to L4Z_LENGTH_PREFIXED because it is recently added and can potentially cause problem if segment creator gets the new version first.
   Let's wait after 0.9 is released and add this in 0.10.0. We should also consider changing the default writer version to 4 which is much more efficient than 2
   
   The V4 chunk writer automatically upgrades `LZ4` to `LZ4_LENGTH_PREFIXED` so I could just change it `LZ4` instead


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org