[GitHub] [pinot] xiangfu0 opened a new pull request, #11290: Fixing the flaky test TupleSketchIntegrationTest

2023-08-07 Thread via GitHub
xiangfu0 opened a new pull request, #11290: URL: https://github.com/apache/pinot/pull/11290 The sketch bytes size can be 1740 or 1756. -- 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

[pinot] 01/01: Fixing the flaky test TupleSketchIntegrationTest

2023-08-07 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch flaky-test-TupleSketchIntegrationTest in repository https://gitbox.apache.org/repos/asf/pinot.git commit 351f6c695e369d1eab28fb67a37dd2620975806e Author: Xiang Fu AuthorDate: Mon Aug 7 23:52:40

[pinot] branch flaky-test-TupleSketchIntegrationTest created (now 351f6c695e)

2023-08-07 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch flaky-test-TupleSketchIntegrationTest in repository https://gitbox.apache.org/repos/asf/pinot.git at 351f6c695e Fixing the flaky test TupleSketchIntegrationTest This branch includes the f

[GitHub] [pinot] abhioncbr closed pull request #11277: Remove commons-httpclient 3.1 library dependency.

2023-08-07 Thread via GitHub
abhioncbr closed pull request #11277: Remove commons-httpclient 3.1 library dependency. URL: https://github.com/apache/pinot/pull/11277 -- 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

[pinot] branch master updated: Fix Java8 compat in Vector Transform tests (#11289)

2023-08-07 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli 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 a6e23c1f73 Fix Java8 compat in Vector Transform tes

[GitHub] [pinot] jackjlli merged pull request #11289: Fix Java8 compat in Vector Transform tests

2023-08-07 Thread via GitHub
jackjlli merged PR #11289: URL: https://github.com/apache/pinot/pull/11289 -- 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.apa

[GitHub] [pinot] dinoocch opened a new pull request, #11289: Fix Java8 compat in Vector Transform tests

2023-08-07 Thread via GitHub
dinoocch opened a new pull request, #11289: URL: https://github.com/apache/pinot/pull/11289 This commit replaces the usage of `"".repeat(...)` added in #11268 with java8 compatible invocations -- This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [pinot] walterddr commented on issue #11272: [multistage] enhance physical plan explain

2023-08-07 Thread via GitHub
walterddr commented on issue #11272: URL: https://github.com/apache/pinot/issues/11272#issuecomment-1668662315 thank you @abhioncbr this is not urgent. -- 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 g

[pinot] branch master updated: Add a method before server starts serving queries (#11288)

2023-08-07 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 f4e1ce9110 Add a method before server starts serv

[GitHub] [pinot] Jackie-Jiang merged pull request #11288: Add a method before server starts serving queries

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

[GitHub] [pinot] Jackie-Jiang opened a new pull request, #11288: Add a method before server starts serving queries

2023-08-07 Thread via GitHub
Jackie-Jiang opened a new pull request, #11288: URL: https://github.com/apache/pinot/pull/11288 Allow custom server starter to perform operations before server starts serving queries -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[pinot] branch master updated: Make PinotTableRestletResourceTest java8 compatible (#11287)

2023-08-07 Thread mcvsubbu
This is an automated email from the ASF dual-hosted git repository. mcvsubbu 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 bc4a225428 Make PinotTableRestletResourceTest j

[GitHub] [pinot] mcvsubbu merged pull request #11287: Make PinotTableRestletResourceTest java8 compatible

2023-08-07 Thread via GitHub
mcvsubbu merged PR #11287: URL: https://github.com/apache/pinot/pull/11287 -- 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.apa

[GitHub] [pinot] Jackie-Jiang commented on issue #11267: Random incorrect result by the query when using CASE statement

2023-08-07 Thread via GitHub
Jackie-Jiang commented on issue #11267: URL: https://github.com/apache/pinot/issues/11267#issuecomment-1668601629 Can you share more details of the expected result and unexpected result? Also, if possible can you try it with the latest release or current master to verify if the issue stil

[GitHub] [pinot] codecov-commenter commented on pull request #11287: Make PinotTableRestletResourceTest java8 compatible

2023-08-07 Thread via GitHub
codecov-commenter commented on PR #11287: URL: https://github.com/apache/pinot/pull/11287#issuecomment-1668596822 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/11287?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) R

[GitHub] [pinot] abhioncbr commented on issue #11272: [multistage] enhance physical plan explain

2023-08-07 Thread via GitHub
abhioncbr commented on issue #11272: URL: https://github.com/apache/pinot/issues/11272#issuecomment-1668558136 I can look into it if not urgent. -- 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 th

[GitHub] [pinot] abhioncbr commented on issue #11251: Create a separated query endpoint for multi-stage query engine

2023-08-07 Thread via GitHub
abhioncbr commented on issue #11251: URL: https://github.com/apache/pinot/issues/11251#issuecomment-1668557693 Please assign it to me if not urgent. -- 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 t

[GitHub] [pinot] dinoocch opened a new pull request, #11287: Make PinotTableRestletResourceTest java8 compatible

2023-08-07 Thread via GitHub
dinoocch opened a new pull request, #11287: URL: https://github.com/apache/pinot/pull/11287 This commit fixes java8 build compatibility that is broken in #11017 Specifically, `Map.of` is only available in java 9+. In this case, this code: ```java TableConfigBuilder(Tab

[pinot] branch master updated: Support parsing ARRAY literal in multistage query engine (#11268)

2023-08-07 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu 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 48490933de Support parsing ARRAY literal in mult

[GitHub] [pinot] xiangfu0 merged pull request #11268: Support parsing ARRAY Literal in a query

2023-08-07 Thread via GitHub
xiangfu0 merged PR #11268: URL: https://github.com/apache/pinot/pull/11268 -- 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.apa

[pinot] branch master updated: fix pinot doap git repo (#11286)

2023-08-07 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu 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 f7d55c5d92 fix pinot doap git repo (#11286) f7d5

[GitHub] [pinot] xiangfu0 merged pull request #11286: fix pinot doap git repo

2023-08-07 Thread via GitHub
xiangfu0 merged PR #11286: URL: https://github.com/apache/pinot/pull/11286 -- 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.apa

[GitHub] [pinot] xiangfu0 opened a new pull request, #11286: fix pinot doap git repo

2023-08-07 Thread via GitHub
xiangfu0 opened a new pull request, #11286: URL: https://github.com/apache/pinot/pull/11286 Fixing the git url -- 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 unsubscrib

[GitHub] [pinot] codecov-commenter commented on pull request #11285: Remove presto jdk8 related dependencies

2023-08-07 Thread via GitHub
codecov-commenter commented on PR #11285: URL: https://github.com/apache/pinot/pull/11285#issuecomment-1668489309 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/11285?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) R

[pinot] branch master updated: Adding ASF DOAP file (#11284)

2023-08-07 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu 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 c1e3159b02 Adding ASF DOAP file (#11284) c1e3159

[GitHub] [pinot] xiangfu0 merged pull request #11284: Adding ASF DOAP file

2023-08-07 Thread via GitHub
xiangfu0 merged PR #11284: URL: https://github.com/apache/pinot/pull/11284 -- 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.apa

[GitHub] [pinot] xiangfu0 commented on issue #11280: Where to get pinot-spark-3-connector-0.13.0-SNAPSHOT-shaded.jar

2023-08-07 Thread via GitHub
xiangfu0 commented on issue #11280: URL: https://github.com/apache/pinot/issues/11280#issuecomment-1668447264 You can build/use it locally by running : `mvn clean package -DskipTests -Pbin-dist -T1C`. The SNAPSHOT jar is not published to maven. Another alternative is to use jitpack

[GitHub] [pinot] xiangfu0 opened a new pull request, #11285: Remove presto jdk8 related dependencies

2023-08-07 Thread via GitHub
xiangfu0 opened a new pull request, #11285: URL: https://github.com/apache/pinot/pull/11285 - Remove jdk8 libs - Remove presto related CI -- 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 sp

[GitHub] [pinot] MeihanLi commented on a diff in pull request #11273: Add Server API to get tenant pools

2023-08-07 Thread via GitHub
MeihanLi commented on code in PR #11273: URL: https://github.com/apache/pinot/pull/11273#discussion_r1286277585 ## pinot-server/src/main/java/org/apache/pinot/server/api/resources/InstanceResource.java: ## @@ -73,4 +75,17 @@ public List getInstanceTags() { } return Col

[GitHub] [pinot] xiangfu0 opened a new pull request, #11284: Adding ASF DOAP file

2023-08-07 Thread via GitHub
xiangfu0 opened a new pull request, #11284: URL: https://github.com/apache/pinot/pull/11284 Adding asf doap file to fill up this page: https://projects.apache.org/project.html?pinot ref from Kafka: https://projects.apache.org/project.html?kafka -- This is an automated message

[GitHub] [pinot] jasperjiaguo commented on a diff in pull request #11200: Add instrumentation for dcount merge. Release bitmaps for and/or doc id set to save memory. `performance`

2023-08-07 Thread via GitHub
jasperjiaguo commented on code in PR #11200: URL: https://github.com/apache/pinot/pull/11200#discussion_r1286251914 ## pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/BaseDistinctAggregateAggregationFunction.java: ## @@ -112,6 +113,9 @@ public Set merge

[GitHub] [pinot] jasperjiaguo commented on a diff in pull request #11200: Add instrumentation for dcount merge. Release bitmaps for and/or doc id set to save memory. `performance`

2023-08-07 Thread via GitHub
jasperjiaguo commented on code in PR #11200: URL: https://github.com/apache/pinot/pull/11200#discussion_r1286251914 ## pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/BaseDistinctAggregateAggregationFunction.java: ## @@ -112,6 +113,9 @@ public Set merge

[GitHub] [pinot] ankitsultana commented on a diff in pull request #11273: Add Server API to get tenant pools

2023-08-07 Thread via GitHub
ankitsultana commented on code in PR #11273: URL: https://github.com/apache/pinot/pull/11273#discussion_r1286207981 ## pinot-server/src/main/java/org/apache/pinot/server/api/resources/InstanceResource.java: ## @@ -73,4 +75,17 @@ public List getInstanceTags() { } return

[pinot] branch master updated: UI: fix sql query options seperator (#10770)

2023-08-07 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 8bc4febba0 UI: fix sql query options seperator (#

[GitHub] [pinot] jayeshchoudhary commented on pull request #10770: UI: fix sql query options seperator

2023-08-07 Thread via GitHub
jayeshchoudhary commented on PR #10770: URL: https://github.com/apache/pinot/pull/10770#issuecomment-1668309668 @Jackie-Jiang @saurabhd336 -- 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

[GitHub] [pinot] Jackie-Jiang merged pull request #10770: UI: fix sql query options seperator

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

[GitHub] [pinot] gortiz commented on pull request #11283: Add spotbugs to detect some cases when plugins use forbidden apis

2023-08-07 Thread via GitHub
gortiz commented on PR #11283: URL: https://github.com/apache/pinot/pull/11283#issuecomment-1668220138 You can see how the error is printed in the GA execution: https://github.com/apache/pinot/actions/runs/5787513059/job/15684598030?pr=11283 -- This is an automated message from the Apache

[GitHub] [pinot] gortiz opened a new pull request, #11283: Add spotbugs to detect some cases when plugins use forbidden apis

2023-08-07 Thread via GitHub
gortiz opened a new pull request, #11283: URL: https://github.com/apache/pinot/pull/11283 This PR includes adds spotbugs execution in the plugin section. The spotbugs configuration ignores all default detectors (we can discuss later if we want to use them in future) but enables a new detect

[GitHub] [pinot] t0mpere opened a new issue, #11282: Minion Batch ingestion scheduling bottleneck

2023-08-07 Thread via GitHub
t0mpere opened a new issue, #11282: URL: https://github.com/apache/pinot/issues/11282 Hello, I've tried to debug why scheduling `SegmentGenerationAndTarPush` Minion jobs take so long to schedule and I've narrowed it down the problem to this part of the code. https://github.com/apac

[GitHub] [pinot] shounakmk219 commented on a diff in pull request #11128: Tenant rebalance and status tracking APIs

2023-08-07 Thread via GitHub
shounakmk219 commented on code in PR #11128: URL: https://github.com/apache/pinot/pull/11128#discussion_r1285724684 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/RebalanceContext.java: ## @@ -0,0 +1,137 @@ +/** + * Licensed to the Apache Soft

[pinot] branch dependabot/maven/jersey.version-3.1.3 created (now 15260dbaea)

2023-08-07 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/jersey.version-3.1.3 in repository https://gitbox.apache.org/repos/asf/pinot.git at 15260dbaea Bump jersey.version from 2.39 to 3.1.3 No new revisions were added by th

[GitHub] [pinot] dependabot[bot] opened a new pull request, #11281: Bump jersey.version from 2.39 to 3.1.3

2023-08-07 Thread via GitHub
dependabot[bot] opened a new pull request, #11281: URL: https://github.com/apache/pinot/pull/11281 Bumps `jersey.version` from 2.39 to 3.1.3. Updates `org.glassfish.jersey.containers:jersey-container-grizzly2-http` from 2.39 to 3.1.3 Updates `org.glassfish.jersey.core:jersey-server

[GitHub] [pinot] shounakmk219 commented on a diff in pull request #11128: Tenant rebalance and status tracking APIs

2023-08-07 Thread via GitHub
shounakmk219 commented on code in PR #11128: URL: https://github.com/apache/pinot/pull/11128#discussion_r1285714926 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/RebalanceContext.java: ## @@ -0,0 +1,137 @@ +/** + * Licensed to the Apache Soft

[GitHub] [pinot] gortiz commented on pull request #11264: Remove JsonUtils dependency due to classpath issues

2023-08-07 Thread via GitHub
gortiz commented on PR #11264: URL: https://github.com/apache/pinot/pull/11264#issuecomment-1667415680 There are two different issues: - We have JsonUtils several times in the classpath with the same package. Luckily, the correct one is loaded first. - The plugins shade jackson using t

[GitHub] [pinot] gortiz commented on a diff in pull request #11205: [Draft] Remove noop

2023-08-07 Thread via GitHub
gortiz commented on code in PR #11205: URL: https://github.com/apache/pinot/pull/11205#discussion_r1285466350 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/executor/OpChainSchedulerService.java: ## @@ -31,151 +33,84 @@ import org.slf4j.LoggerFactory; -

[GitHub] [pinot] gortiz commented on a diff in pull request #11205: [Draft] Remove noop

2023-08-07 Thread via GitHub
gortiz commented on code in PR #11205: URL: https://github.com/apache/pinot/pull/11205#discussion_r1285457564 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/mailbox/ReceivingMailbox.java: ## @@ -51,16 +51,30 @@ public class ReceivingMailbox { // TODO: Revisit if