Re: [PR] For consuming segment, avoid using setter in IndexLoadingConfig [pinot]

2024-10-09 Thread via GitHub
Jackie-Jiang merged PR #14190: URL: https://github.com/apache/pinot/pull/14190 -- 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] For consuming segment, avoid using setter in IndexLoadingConfig [pinot]

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

[PR] For consuming segment, avoid using setter in IndexLoadingConfig [pinot]

2024-10-08 Thread via GitHub
Jackie-Jiang opened a new pull request, #14190: URL: https://github.com/apache/pinot/pull/14190 Sub-task of #14106 Currently in `RealtimeSegmentDataManager` we rely on calling `IndexLoadingConfig.addInvertedIndexColumns()` to create inverted index for sorted column. It can cause inc