Re: [PR] Fix segment size check in OfflineClusterIntegrationTest [pinot]

2024-06-13 Thread via GitHub
Jackie-Jiang merged PR #13389: URL: https://github.com/apache/pinot/pull/13389 -- 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

Re: [PR] Fix segment size check in OfflineClusterIntegrationTest [pinot]

2024-06-13 Thread via GitHub
codecov-commenter commented on PR #13389: URL: https://github.com/apache/pinot/pull/13389#issuecomment-2166931380 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/13389?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

[PR] Fix segment size check in OfflineClusterIntegrationTest [pinot]

2024-06-13 Thread via GitHub
Jackie-Jiang opened a new pull request, #13389: URL: https://github.com/apache/pinot/pull/13389 Segment size is platform dependent because of the native library used by the chunk compressor. Do not use the hard-coded segment size. Cache the segment size of the generated segments. -- T