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

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

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

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

2024-12-06 Thread via GitHub
Jackie-Jiang opened a new pull request, #14618: URL: https://github.com/apache/pinot/pull/14618 #14158 introduced the query option validation, but some of the validations are too strict and some are too loose. This PR changes the validations of the following query options: - NUM_R