Re: [PR] [DRAFT] custom partial upsert row merger [pinot]

2024-09-05 Thread via GitHub
rohityadav1993 closed pull request #11584: [DRAFT] custom partial upsert row merger URL: https://github.com/apache/pinot/pull/11584 -- 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 comme

Re: [PR] [Backfill] flink based upsert table backfill support [pinot]

2024-09-05 Thread via GitHub
klsince commented on code in PR #13837: URL: https://github.com/apache/pinot/pull/13837#discussion_r1746526919 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/IngestionUtils.java: ## @@ -169,6 +170,16 @@ private static SegmentNameGenerator getSegmentNa

Re: [PR] Add support for configuring aggregation function parameters in the star-tree index [pinot]

2024-09-05 Thread via GitHub
yashmayya commented on code in PR #13835: URL: https://github.com/apache/pinot/pull/13835#discussion_r1746502018 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/aggregator/DistinctCountCPCSketchValueAggregator.java: ## @@ -34,11 +35,11 @@ public class Distinc

(pinot) branch master updated: In SchemaConformingTransformer, Flatten array into multiple entries each with a key and array value. (#13890)

2024-09-05 Thread tingchen
This is an automated email from the ASF dual-hosted git repository. tingchen 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 7ddb7a46f6 In SchemaConformingTransformer, Flat

Re: [PR] Part-1: Pinot Timeseries Engine SPI [pinot]

2024-09-05 Thread via GitHub
Jackie-Jiang commented on code in PR #13885: URL: https://github.com/apache/pinot/pull/13885#discussion_r1746027496 ## pinot-timeseries/pinot-timeseries-spi/src/main/java/org/apache/pinot/tsdb/spi/PinotTimeSeriesConfiguration.java: ## @@ -0,0 +1,57 @@ +/** + * Licensed to the Ap

(pinot) branch master updated: Bump io.netty:netty-bom from 4.1.112.Final to 4.1.113.Final (#13940)

2024-09-05 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 bc4e3a3613 Bump io.netty:netty-bom from 4.1.112.F

Re: [PR] Bump io.netty:netty-bom from 4.1.112.Final to 4.1.113.Final [pinot]

2024-09-05 Thread via GitHub
Jackie-Jiang merged PR #13940: URL: https://github.com/apache/pinot/pull/13940 -- 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] [Spark Connector] Write support for creating Pinot segments [pinot]

2024-09-05 Thread via GitHub
cbalci commented on code in PR #13748: URL: https://github.com/apache/pinot/pull/13748#discussion_r1745988697 ## .github/workflows/pinot_tests.yml: ## @@ -123,6 +123,11 @@ jobs: --add-exports=jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED --add-exp

Re: [PR] colocated-without-hints: First commit with the proposed code [pinot]

2024-09-05 Thread via GitHub
ankitsultana commented on PR #13943: URL: https://github.com/apache/pinot/pull/13943#issuecomment-2332383406 @gortiz are you aiming to make it so that the entire plan can be colocated when possible? From Uber side we were planning to add this feature back again this year. For our cur

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.27.19 deleted (was 80c38759e8)

2024-09-05 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/software.amazon.awssdk-bom-2.27.19 in repository https://gitbox.apache.org/repos/asf/pinot.git was 80c38759e8 Bump software.amazon.awssdk:bom from 2.27.18 to 2.27.19 The r

Re: [PR] Bump org.checkerframework:checker-qual from 3.46.0 to 3.47.0 [pinot]

2024-09-05 Thread via GitHub
Jackie-Jiang merged PR #13938: URL: https://github.com/apache/pinot/pull/13938 -- 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] Multi stage explain [pinot]

2024-09-05 Thread via GitHub
yashmayya commented on code in PR #13733: URL: https://github.com/apache/pinot/pull/13733#discussion_r1741452894 ## pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java: ## @@ -1192,6 +1209,9 @@ public enum WindowOverFlowMode { public static class PlanVer

Re: [I] MinionTenant as a table config [pinot]

2024-09-05 Thread via GitHub
tarun11Mavani commented on issue #12698: URL: https://github.com/apache/pinot/issues/12698#issuecomment-2331638479 @tibrewalpratik17 created a issue for Add validation of instance tag provided in task-config in TableConfigUtils. Please link the same to this issue description. https:

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.27.19 created (now 80c38759e8)

2024-09-05 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/software.amazon.awssdk-bom-2.27.19 in repository https://gitbox.apache.org/repos/asf/pinot.git at 80c38759e8 Bump software.amazon.awssdk:bom from 2.27.18 to 2.27.19 N

Re: [PR] Add broker API to run a query on both query engines and compare results [pinot]

2024-09-05 Thread via GitHub
vrajat commented on code in PR #13746: URL: https://github.com/apache/pinot/pull/13746#discussion_r1745157640 ## pinot-broker/src/main/java/org/apache/pinot/broker/api/resources/PinotClientRequest.java: ## @@ -236,6 +243,76 @@ public void processSqlWithMultiStageQueryEnginePost(