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-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
