Re: [I] [Pinot 1.2.0 UI] No support for "database" construct from UI [pinot]

2025-05-27 Thread via GitHub
NihalJain commented on issue #14068: URL: https://github.com/apache/pinot/issues/14068#issuecomment-2915013770 > What is the usecase for showing tables/schemas across all databases in one view? We want to do this because as an ADMIN i may want a single view of the cluster and all its

Re: [I] GET /logicalTables ignores database parameter. [pinot]

2025-05-27 Thread via GitHub
NihalJain commented on issue #15908: URL: https://github.com/apache/pinot/issues/15908#issuecomment-2914995973 I just created https://github.com/apache/pinot/issues/15918 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[I] Enhance REST API to support fetching resources for all databases for all relevant APIs [pinot]

2025-05-27 Thread via GitHub
NihalJain opened a new issue, #15918: URL: https://github.com/apache/pinot/issues/15918 Currently pinot allows us to fetch tables specific to a DB only, but there can be use case where users/admins may want to see all list of DBs across the cluster. For example we want to view all tables i

Re: [I] GET /logicalTables ignores database parameter. [pinot]

2025-05-27 Thread via GitHub
NihalJain commented on issue #15908: URL: https://github.com/apache/pinot/issues/15908#issuecomment-2914913875 HI @vrajat I had a similar use case propose at https://github.com/apache/pinot/issues/14068#issuecomment-2464606418 Do you mind having a look? -- This is an automated message

(pinot) branch master updated: ASOF JOIN (#15630)

2025-05-27 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya 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 767cb9b8a1 ASOF JOIN (#15630) 767cb9b8a1 is de

Re: [I] GET /logicalTables ignores database parameter. [pinot]

2025-05-27 Thread via GitHub
abhishekbafna commented on issue #15908: URL: https://github.com/apache/pinot/issues/15908#issuecomment-2914892894 @vrajat Both the table and schema GET APIs return resources/entities specific to either the default database or the one specified in the headers. These APIs don’t support fetch

Re: [PR] ASOF JOIN [pinot]

2025-05-27 Thread via GitHub
yashmayya merged PR #15630: URL: https://github.com/apache/pinot/pull/15630 -- 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.ap

Re: [PR] Reduce ZK access for SendStatsPredicate [pinot]

2025-05-27 Thread via GitHub
mcvsubbu commented on PR #15895: URL: https://github.com/apache/pinot/pull/15895#issuecomment-2914467426 > @mcvsubbu To workaround the overhead of watches, you can manually set `pinot.query.mse.stats.mode` to `ALWAYS` or `NEVER` I am guessing that both of these have a cost, and I am t

Re: [PR] [WIP] Tenant Rebalance Improvement [pinot]

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

Re: [PR] Modify OFFLINE and REALTIME completed segments to calculate default partitionId based on same logic as consuming segments rather than 0 [pinot]

2025-05-27 Thread via GitHub
somandal commented on PR #15817: URL: https://github.com/apache/pinot/pull/15817#issuecomment-2914332159 > * When there is single partition (`numPartitions == 1`), return 0 > * For RT segment following LLC segment name, use the one from segment name > * When `partitionColumn` is not nul

Re: [PR] Modify OFFLINE and REALTIME completed segments to calculate default partitionId based on same logic as consuming segments rather than 0 [pinot]

2025-05-27 Thread via GitHub
somandal commented on PR #15817: URL: https://github.com/apache/pinot/pull/15817#issuecomment-2914307933 Yet to add test -- 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

[PR] Reduce ZK access for SendStatsPredicate (#15895) [pinot]

2025-05-27 Thread via GitHub
praveenc7 opened a new pull request, #15917: URL: https://github.com/apache/pinot/pull/15917 HotFix for https://github.com/apache/pinot/issues/15890 -- 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] Modify OFFLINE and REALTIME completed segments to calculate default partitionId based on same logic as consuming segments rather than 0 [pinot]

2025-05-27 Thread via GitHub
somandal commented on code in PR #15817: URL: https://github.com/apache/pinot/pull/15817#discussion_r2110394455 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/segment/SegmentAssignmentUtils.java: ## @@ -448,10 +472,10 @@ public static int get

Re: [PR] Reduce ZK access for SendStatsPredicate (#15895) [pinot]

2025-05-27 Thread via GitHub
sajjad-moradi merged PR #15917: URL: https://github.com/apache/pinot/pull/15917 -- 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...@pino

(pinot) branch hotfix-zk-watcher-2 updated: Reduce ZK access for SendStatsPredicate (#15895) (#15917)

2025-05-27 Thread sajjad
This is an automated email from the ASF dual-hosted git repository. sajjad pushed a commit to branch hotfix-zk-watcher-2 in repository https://gitbox.apache.org/repos/asf/pinot.git The following commit(s) were added to refs/heads/hotfix-zk-watcher-2 by this push: new 4fa09f6163 Reduce ZK a

(pinot) branch hotfix-zk-watcher-2 created (now 1fdeadd6ef)

2025-05-27 Thread vvivekiyer
This is an automated email from the ASF dual-hosted git repository. vvivekiyer pushed a change to branch hotfix-zk-watcher-2 in repository https://gitbox.apache.org/repos/asf/pinot.git at 1fdeadd6ef [Minor] Make all constants in public access (#15605) No new revisions were added by this

(pinot) branch hotfix-zk-watcher created (now ba48afd309)

2025-05-27 Thread sajjad
This is an automated email from the ASF dual-hosted git repository. sajjad pushed a change to branch hotfix-zk-watcher in repository https://gitbox.apache.org/repos/asf/pinot.git at ba48afd309 Enable Correct GROUP BY-Only Processing with accurateGroupByWithoutOrderBy (#15844) No new revi

(pinot) branch hotfix-zk-watcher updated: Reduce ZK access for SendStatsPredicate (#15895) (#15916)

2025-05-27 Thread sajjad
This is an automated email from the ASF dual-hosted git repository. sajjad pushed a commit to branch hotfix-zk-watcher in repository https://gitbox.apache.org/repos/asf/pinot.git The following commit(s) were added to refs/heads/hotfix-zk-watcher by this push: new e6482797f6 Reduce ZK access

Re: [PR] Hotfix ZK Watcher issue - NOT ON MASTER BRANCH [pinot]

2025-05-27 Thread via GitHub
sajjad-moradi merged PR #15916: URL: https://github.com/apache/pinot/pull/15916 -- 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...@pino

[PR] Hotfix ZK Watcher issue - NOT ON MASTER BRANCH [pinot]

2025-05-27 Thread via GitHub
sajjad-moradi opened a new pull request, #15916: URL: https://github.com/apache/pinot/pull/15916 (no comment) -- 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

Re: [PR] Reduce ZK access for SendStatsPredicate [pinot]

2025-05-27 Thread via GitHub
Jackie-Jiang commented on PR #15895: URL: https://github.com/apache/pinot/pull/15895#issuecomment-2914040443 @mcvsubbu To workaround the overhead of watches, you can manually set `pinot.query.mse.stats.mode` to `ALWAYS` or `NEVER` -- This is an automated message from the Apache Git Servi

Re: [PR] Reduce ZK access for SendStatsPredicate (#15895) [pinot]

2025-05-27 Thread via GitHub
praveenc7 closed pull request #15915: Reduce ZK access for SendStatsPredicate (#15895) URL: https://github.com/apache/pinot/pull/15915 -- 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 co

[PR] Reduce ZK access for SendStatsPredicate (#15895) [pinot]

2025-05-27 Thread via GitHub
praveenc7 opened a new pull request, #15915: URL: https://github.com/apache/pinot/pull/15915 Instructions: 1. The PR has to be tagged with at least one of the following labels (*): 1. `feature` 2. `bugfix` 3. `performance` 4. `ui` 5. `backward-incompat`

Re: [PR] Reduce ZK access for SendStatsPredicate [pinot]

2025-05-27 Thread via GitHub
mcvsubbu commented on PR #15895: URL: https://github.com/apache/pinot/pull/15895#issuecomment-2914031858 I suggest looking at mechanisms other than watches. It may be slower to react, but at least it wont cause production outages due to zookeeper falling over. -- This is an automated me

Re: [PR] Reduce ZK access for SendStatsPredicate [pinot]

2025-05-27 Thread via GitHub
Jackie-Jiang commented on PR #15895: URL: https://github.com/apache/pinot/pull/15895#issuecomment-2914011383 This mechanism is introduced to tolerate backward incompatible changes for query stats. @gortiz Do you foresee future changes that might be backward incompatible? Given this is addin

Re: [I] [multistage] Sub-Optimal Plan for IN Clause When Used with OR [pinot]

2025-05-27 Thread via GitHub
Jackie-Jiang commented on issue #15442: URL: https://github.com/apache/pinot/issues/15442#issuecomment-2913895306 @ankitsultana We have upgraded Calcite to `1.39`. Can you help confirm if the issue is solved? -- This is an automated message from the Apache Git Service. To respond to the m

Re: [PR] Handles server's rate limit config change without restart [pinot]

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

Re: [I] Add a rest API to remove ingestion metrics [pinot]

2025-05-27 Thread via GitHub
itschrispeck commented on issue #15894: URL: https://github.com/apache/pinot/issues/15894#issuecomment-2913958399 > We should also validate if rebalance including consuming segment can properly remove the ingestion metrics Adding a data point, we often see issues with a stale `llc_pa

[PR] Handles server's rate limit config change without restart [pinot]

2025-05-27 Thread via GitHub
noob-se7en opened a new pull request, #15914: URL: https://github.com/apache/pinot/pull/15914 Instructions: 1. The PR has to be tagged with at least one of the following labels (*): 1. `feature` 2. `bugfix` 3. `performance` 4. `ui` 5. `backward-incompat`

Re: [PR] Apply BOM to pin transitive dependencie [pinot]

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

(pinot) branch dependabot/maven/org.scala-lang.modules-scala-xml_2.12-2.4.0 deleted (was 60cd08093b)

2025-05-27 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/org.scala-lang.modules-scala-xml_2.12-2.4.0 in repository https://gitbox.apache.org/repos/asf/pinot.git was 60cd08093b Bump org.scala-lang.modules:scala-xml_2.12 from 2.3.0

(pinot) branch master updated (a61acb71b6 -> ba48afd309)

2025-05-27 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 a61acb71b6 [bugfix] Remove deleting segments from table status info (#15725) add ba48afd309 Enable Correct GROUP

Re: [PR] Enable Correct GROUP BY-Only Processing with accurateGroupByWithoutOrderBy [pinot]

2025-05-27 Thread via GitHub
Jackie-Jiang merged PR #15844: URL: https://github.com/apache/pinot/pull/15844 -- 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] Reduce ZK access for SendStatsPredicate [pinot]

2025-05-27 Thread via GitHub
Jackie-Jiang merged PR #15895: URL: https://github.com/apache/pinot/pull/15895 -- 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

(pinot) branch master updated: [bugfix] Remove deleting segments from table status info (#15725)

2025-05-27 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 a61acb71b6 [bugfix] Remove deleting segments from

Re: [PR] [bugfix] Remove deleting segments from table status info [pinot]

2025-05-27 Thread via GitHub
Jackie-Jiang merged PR #15725: URL: https://github.com/apache/pinot/pull/15725 -- 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

(pinot) branch master updated: Reduce ZK access for SendStatsPredicate (#15895)

2025-05-27 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 38e213d8fa Reduce ZK access for SendStatsPredicat

Re: [I] Pinot-Server : Excessive ZooKeeper Watches on Participant Configs Causing Latency and Ingestion Lag [pinot]

2025-05-27 Thread via GitHub
Jackie-Jiang closed issue #15890: Pinot-Server : Excessive ZooKeeper Watches on Participant Configs Causing Latency and Ingestion Lag URL: https://github.com/apache/pinot/issues/15890 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] Reduce ZK access for SendStatsPredicate [pinot]

2025-05-27 Thread via GitHub
Jackie-Jiang commented on code in PR #15895: URL: https://github.com/apache/pinot/pull/15895#discussion_r2109776831 ## pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SendStatsPredicate.java: ## @@ -67,87 +74,155 @@ public static SendStatsPredicate create(PinotC

Re: [PR] Reduce ZK access for SendStatsPredicate [pinot]

2025-05-27 Thread via GitHub
Jackie-Jiang commented on code in PR #15895: URL: https://github.com/apache/pinot/pull/15895#discussion_r2109774386 ## pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SendStatsPredicate.java: ## @@ -67,87 +74,155 @@ public static SendStatsPredicate create(PinotC

Re: [PR] Reduce ZK access for SendStatsPredicate [pinot]

2025-05-27 Thread via GitHub
praveenc7 commented on code in PR #15895: URL: https://github.com/apache/pinot/pull/15895#discussion_r2109764812 ## pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SendStatsPredicate.java: ## @@ -67,87 +74,155 @@ public static SendStatsPredicate create(PinotConf

Re: [PR] Bump org.scala-lang.modules:scala-xml_2.12 from 2.3.0 to 2.4.0 [pinot]

2025-05-27 Thread via GitHub
Jackie-Jiang merged PR #15912: URL: https://github.com/apache/pinot/pull/15912 -- 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

(pinot) branch master updated (2c9b51b8dc -> f18ac1eb67)

2025-05-27 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 2c9b51b8dc Better mse stats (#15901) add f18ac1eb67 Bump org.scala-lang.modules:scala-xml_2.12 from 2.3.0 to 2.4.

[I] Support Gorilla encoding (Delta of Delta, XOR) Compression Codec [pinot]

2025-05-27 Thread via GitHub
deemoliu opened a new issue, #15913: URL: https://github.com/apache/pinot/issues/15913 (no comment) -- 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:

Re: [I] MSE does not parse Fully-Qualified Table Names [pinot]

2025-05-27 Thread via GitHub
vrajat commented on issue #15911: URL: https://github.com/apache/pinot/issues/15911#issuecomment-2913133778 I had a offline discussion and the actual bug is in SSE (case 2) and not case 1. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] [Bugfix] [UpsertCompactMerge] Resolve data inconsistency in merge compaction caused by replica create time mismatch [pinot]

2025-05-27 Thread via GitHub
tarun11Mavani commented on code in PR #15863: URL: https://github.com/apache/pinot/pull/15863#discussion_r2109498518 ## pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/src/main/java/org/apache/pinot/plugin/minion/tasks/upsertcompactmerge/UpsertCompactMergeTaskGenerator.java:

Re: [PR] MSE: Return partial stats on errors send by servers [pinot]

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

(pinot) branch dependabot/maven/org.scala-lang.modules-scala-xml_2.12-2.4.0 created (now 60cd08093b)

2025-05-27 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/org.scala-lang.modules-scala-xml_2.12-2.4.0 in repository https://gitbox.apache.org/repos/asf/pinot.git at 60cd08093b Bump org.scala-lang.modules:scala-xml_2.12 from 2

[PR] Bump org.scala-lang.modules:scala-xml_2.12 from 2.3.0 to 2.4.0 [pinot]

2025-05-27 Thread via GitHub
dependabot[bot] opened a new pull request, #15912: URL: https://github.com/apache/pinot/pull/15912 Bumps [org.scala-lang.modules:scala-xml_2.12](https://github.com/scala/scala-xml) from 2.3.0 to 2.4.0. Release notes Sourced from https://github.com/scala/scala-xml/releases";>org.sc

Re: [PR] Refactor authorization code to reduce duplication and inconsistency [pinot]

2025-05-27 Thread via GitHub
NihalJain commented on PR #15861: URL: https://github.com/apache/pinot/pull/15861#issuecomment-2912139771 > Making sure I understand the goal correctly: > > * Common code for each of controller, broker & server have been moved to abstract classes in each of the packages. > * Common

Re: [PR] Refactor authorization code to reduce duplication and inconsistency [pinot]

2025-05-27 Thread via GitHub
NihalJain commented on PR #15861: URL: https://github.com/apache/pinot/pull/15861#issuecomment-2912140454 Also rebased to re-trigger tests! -- 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 spe

Re: [PR] Add sanity checks to ensure write permissions for instance data and s… [pinot]

2025-05-27 Thread via GitHub
NihalJain commented on PR #15876: URL: https://github.com/apache/pinot/pull/15876#issuecomment-2912126540 Rebased to re-trigger tests! -- 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

[I] MSE does not parse Fully-Qualified Table Names [pinot]

2025-05-27 Thread via GitHub
vrajat opened a new issue, #15911: URL: https://github.com/apache/pinot/issues/15911 _Case 1: fully-qualified table name does errors out with MSE. (Error is 190=TableDoesNotExistError)_ ``` ❯ curl --request POST http://localhost:8000/query/sql -d '{"sql": "select count(*) from meesho

(pinot) branch master updated: Better mse stats (#15901)

2025-05-27 Thread gortiz
This is an automated email from the ASF dual-hosted git repository. gortiz 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 2c9b51b8dc Better mse stats (#15901) 2c9b51b8dc i

Re: [PR] Better mse stats [pinot]

2025-05-27 Thread via GitHub
gortiz merged PR #15901: URL: https://github.com/apache/pinot/pull/15901 -- 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.apach

[PR] MSE: Return partial stats on errors send by servers [pinot]

2025-05-27 Thread via GitHub
gortiz opened a new pull request, #15910: URL: https://github.com/apache/pinot/pull/15910 In previous PR #15245, I modified MSE to return partial query stats when errors are detected during query execution. However, that PR didn't change the broker part, so although the broker received the

Re: [I] [UpsertMergeCompactTask] Data inconsistency issue across segment replica due to different segment creation time after enabling UpsertMergeCompactTask [pinot]

2025-05-27 Thread via GitHub
tarun11Mavani commented on issue #15846: URL: https://github.com/apache/pinot/issues/15846#issuecomment-2911837876 @Jackie-Jiang One more point is that, tie breaker in should [shouldReplaceOnComparisonTie](https://github.com/apache/pinot/blob/d6732ffd1943a6a11358ad36645e07769e1c84ff/pinot-se

Re: [PR] [BugFix] Handling NPE in VarianceAggregationFunction merger [pinot]

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

[PR] [BugFix] Handling NPE in VarianceAggregationFunction merger [pinot]

2025-05-27 Thread via GitHub
jitendrakr88 opened a new pull request, #15909: URL: https://github.com/apache/pinot/pull/15909 This change fixes a NPE in the VarianceAggregationFunction merger function by handling nulls regardless of _nullHandlingEnabled, which may be more robust. -- This is an automated message fr

(pinot) branch master updated: Logical table query quota enforcement - SSE (#15839)

2025-05-27 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya 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 d6732ffd19 Logical table query quota enforceme

Re: [I] [Logical Table] Support QPS throttling for logical tables in SSE [pinot]

2025-05-27 Thread via GitHub
yashmayya closed issue #15781: [Logical Table] Support QPS throttling for logical tables in SSE URL: https://github.com/apache/pinot/issues/15781 -- 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

Re: [PR] Logical table query quota enforcement - SSE [pinot]

2025-05-27 Thread via GitHub
yashmayya merged PR #15839: URL: https://github.com/apache/pinot/pull/15839 -- 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.ap

Re: [PR] Change Pinot UI to be able to show json on error and MSE stats if any [pinot]

2025-05-27 Thread via GitHub
gortiz merged PR #15898: URL: https://github.com/apache/pinot/pull/15898 -- 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.apach

(pinot) branch master updated: Change Pinot UI to be able to show json on error and MSE stats if any (#15898)

2025-05-27 Thread gortiz
This is an automated email from the ASF dual-hosted git repository. gortiz 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 009ecc1249 Change Pinot UI to be able to show jso

Re: [PR] Reduce ZK access for SendStatsPredicate [pinot]

2025-05-27 Thread via GitHub
gortiz commented on code in PR #15895: URL: https://github.com/apache/pinot/pull/15895#discussion_r2108348994 ## pinot-server/src/main/java/org/apache/pinot/server/starter/helix/SendStatsPredicate.java: ## @@ -67,87 +74,155 @@ public static SendStatsPredicate create(PinotConfigu