Re: [PR] Allow configurable initial capacity for IndexedTable [pinot]

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

(pinot) branch master updated (e8dcba123a -> 1b87488aea)

2024-12-08 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from e8dcba123a Fix query option validation for group-by queries (#14618) add 1b87488aea Strip Matrix parameter from Ba

Re: [PR] Strip Matrix parameter from BasePath check [pinot]

2024-12-08 Thread via GitHub
Jackie-Jiang merged PR #14383: URL: https://github.com/apache/pinot/pull/14383 -- 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] Adds Support of maxNumRowsPerTask in RealtimeToOfflineSegmentsTasksGe… [pinot]

2024-12-08 Thread via GitHub
Harnoor7 commented on PR #14578: URL: https://github.com/apache/pinot/pull/14578#issuecomment-2527105131 Will update tests once approach seems good to go. -- 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 t

Re: [PR] Strip Matrix parameter from BasePath check [pinot]

2024-12-08 Thread via GitHub
soumitra-st commented on PR #14383: URL: https://github.com/apache/pinot/pull/14383#issuecomment-2527005143 Thanks @praveenc7 for the context in Slack. The gap exposes serious security issue. Approving the PR. @Jackie-Jiang , we can take up the whitelisting enhancement separately. WDY

(pinot) branch master updated: Fix query option validation for group-by queries (#14618)

2024-12-08 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git The following commit(s) were added to refs/heads/master by this push: new e8dcba123a Fix query option validation for group-

Re: [PR] Fix query option validation for group-by queries [pinot]

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