(pinot) branch master updated: Fix bug in logging in UpsertCompaction task (#12419)

2024-02-16 Thread ankitsultana
This is an automated email from the ASF dual-hosted git repository. ankitsultana 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 20f3fc6790 Fix bug in logging in

(pinot) branch master updated: Add upsert-snapshot timer metric (#12383)

2024-02-16 Thread ankitsultana
This is an automated email from the ASF dual-hosted git repository. ankitsultana 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 2a4e79dcf0 Add upsert-snapshot timer

(pinot) branch master updated: Adding support to insecure TLS when creating SSLFactory (#12425)

2024-02-16 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 706dd5926d Adding support to insecure TLS when

(pinot) branch dependabot/maven/com.google.api.grpc-proto-google-iam-v1-1.29.0 deleted (was 880cb591a6)

2024-02-16 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/com.google.api.grpc-proto-google-iam-v1-1.29.0 in repository https://gitbox.apache.org/repos/asf/pinot.git was 880cb591a6 Bump com.google.api.grpc:proto-google-iam-v1

(pinot) branch master updated: Upgrade google library versions (#12432)

2024-02-16 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 b5dc09aeb5 Upgrade google library versions

Re: [I] [partial-upsert] Partial Upsert Table Data Inconsistency Issues [pinot]

2024-02-16 Thread via GitHub
ankitsultana commented on issue #12396: URL: https://github.com/apache/pinot/issues/12396#issuecomment-1949474503 > I'd assume the issue is about during table rebalance, the consuming segment might get started even before the other immutable segments in the same table partition get fully

Re: [I] [partial-upsert] Partial Upsert Table Data Inconsistency Issues [pinot]

2024-02-16 Thread via GitHub
klsince commented on issue #12396: URL: https://github.com/apache/pinot/issues/12396#issuecomment-1949460955 Thanks for reporting the issues. > For the first 2 issues, we need to figure out how to handle partial-upsert with same comparison value. For partial upsert, the insert order

Re: [PR] Fix Bug in Handling Equal Comparison Column Values in Upsert [pinot]

2024-02-16 Thread via GitHub
ankitsultana commented on code in PR #12395: URL: https://github.com/apache/pinot/pull/12395#discussion_r1492976184 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/ConcurrentMapPartitionUpsertMetadataManager.java: ## @@ -69,6 +69,9 @@ protected void

(pinot) branch master updated: Ensure FieldConfig.getEncodingType() is never null (#12430)

2024-02-16 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 21346f6001 Ensure FieldConfig.getEncodingType()

Re: [PR] Ensure FieldConfig.getEncodingType() is never null [pinot]

2024-02-16 Thread via GitHub
Jackie-Jiang merged PR #12430: URL: https://github.com/apache/pinot/pull/12430 -- 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: [PR] Fix Bug in Handling Equal Comparison Column Values in Upsert [pinot]

2024-02-16 Thread via GitHub
tibrewalpratik17 commented on code in PR #12395: URL: https://github.com/apache/pinot/pull/12395#discussion_r1492896625 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/ConcurrentMapPartitionUpsertMetadataManager.java: ## @@ -69,6 +69,9 @@ protected

(pinot) branch master updated: Add metrics for SEGMENTS_WITH_LESS_REPLICAS monitoring (#12336)

2024-02-16 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 9eaa3a1364 Add metrics for

Re: [PR] Add metrics for SEGMENTS_WITH_LESS_REPLICAS monitoring [pinot]

2024-02-16 Thread via GitHub
chenboat merged PR #12336: URL: https://github.com/apache/pinot/pull/12336 -- 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] Add an extraContainerPort array option to k8s helm chart [pinot]

2024-02-16 Thread via GitHub
psheets closed issue #10538: Add an extraContainerPort array option to k8s helm chart URL: https://github.com/apache/pinot/issues/10538 -- 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

Re: [I] Add change streams to tables [pinot]

2024-02-16 Thread via GitHub
psheets closed issue #10573: Add change streams to tables URL: https://github.com/apache/pinot/issues/10573 -- 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] [multistage][feature] FunctionRegistry unification [pinot]

2024-02-16 Thread via GitHub
gortiz commented on code in PR #12302: URL: https://github.com/apache/pinot/pull/12302#discussion_r1492748325 ## pinot-common/src/main/java/org/apache/pinot/common/function/FunctionRegistry.java: ## @@ -94,83 +155,196 @@ private FunctionRegistry() { public static void init()

Re: [PR] [multistage][feature] FunctionRegistry unification [pinot]

2024-02-16 Thread via GitHub
gortiz commented on code in PR #12302: URL: https://github.com/apache/pinot/pull/12302#discussion_r1492748325 ## pinot-common/src/main/java/org/apache/pinot/common/function/FunctionRegistry.java: ## @@ -94,83 +155,196 @@ private FunctionRegistry() { public static void init()

Re: [I] Invalid transform function JSONPATHDOUBLE [pinot]

2024-02-16 Thread via GitHub
abhioncbr commented on issue #12422: URL: https://github.com/apache/pinot/issues/12422#issuecomment-1948815582 Can you check the ponot version is same on docker and on K8s? Which Pinot version? -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [I] commons-configuration2 performance issue during segment metadata load [pinot]

2024-02-16 Thread via GitHub
abhioncbr commented on issue #12433: URL: https://github.com/apache/pinot/issues/12433#issuecomment-1948791380 @xiangfu0 this is more about skipping unnecessary escaping of the segment metadata keys to make initialization faster, along with the introduction of versioning of segment

Re: [PR] Change helm app version to 1.0.0 for Apache Pinot latest release version [pinot]

2024-02-16 Thread via GitHub
codecov-commenter commented on PR #12436: URL: https://github.com/apache/pinot/pull/12436#issuecomment-1948523623 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/12436?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report All modified

[PR] Change helm app version to 1.0.0 for Apache Pinot latest release version [pinot]

2024-02-16 Thread via GitHub
xiangfu0 opened a new pull request, #12436: URL: https://github.com/apache/pinot/pull/12436 Change the Helm App version to match Apache Pinot release version. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

(pinot-site) 01/01: Initial empty commit

2024-02-16 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch new-site-dev in repository https://gitbox.apache.org/repos/asf/pinot-site.git commit 93309a07c342213fd33a3850cdace04ceb251187 Author: Xiang Fu AuthorDate: Fri Feb 16 19:08:48 2024 +0800

(pinot-site) branch new-site-dev created (now 67dc7484)

2024-02-16 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch new-site-dev in repository https://gitbox.apache.org/repos/asf/pinot-site.git at 67dc7484 Initial empty commit This branch includes the following new commits: new 67dc7484 Initial

(pinot-site) branch new-site-dev updated (67dc7484 -> 93309a07)

2024-02-16 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch new-site-dev in repository https://gitbox.apache.org/repos/asf/pinot-site.git discard 67dc7484 Initial empty commit new 93309a07 Initial empty commit This update added new revisions

(pinot-site) 01/01: Initial empty commit

2024-02-16 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a commit to branch new-site-dev in repository https://gitbox.apache.org/repos/asf/pinot-site.git commit 67dc7484064bb50f30b89521d577df2bb7dddfa1 Author: Xiang Fu AuthorDate: Fri Feb 16 19:08:48 2024 +0800

Re: [PR] [draft] Mode null benchmark [pinot]

2024-02-16 Thread via GitHub
gortiz commented on PR #12354: URL: https://github.com/apache/pinot/pull/12354#issuecomment-1948171973 I've repeated the benchmark with: - Java 21 - 1M rows per segment (instead of 10k) - 3 consecutive nulls every 3 not null rows (instead of 1 every 127) And the results are:

Re: [PR] bumped calcite-core to 1.36.0 [pinot]

2024-02-16 Thread via GitHub
xiangfu0 commented on PR #12364: URL: https://github.com/apache/pinot/pull/12364#issuecomment-1948160887 I think there are a lot of API changes between 1.30 and 1.36. It's not that easy to upgrade Calcite. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [draft] Mode null benchmark [pinot]

2024-02-16 Thread via GitHub
gortiz commented on PR #12354: URL: https://github.com/apache/pinot/pull/12354#issuecomment-1948107488 We could try to execute the same benchmark changing the null distribution and increasing the size of the segments, but TBH the difference seems very small when we compare with real costs

Re: [PR] [draft] Mode null benchmark [pinot]

2024-02-16 Thread via GitHub
gortiz commented on PR #12354: URL: https://github.com/apache/pinot/pull/12354#issuecomment-1948104199 I've applied similar changes in SUM and results are: In AMD Ryzen 9 3900X, Java 11: ``` Benchmark (_aQueryTemplate)

Re: [PR] [draft] Mode null benchmark [pinot]

2024-02-16 Thread via GitHub
codecov-commenter commented on PR #12354: URL: https://github.com/apache/pinot/pull/12354#issuecomment-1948072486 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/12354?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report Attention:

[I] Error in loading `Cluster Manager` page [pinot]

2024-02-16 Thread via GitHub
xiangfu0 opened a new issue, #12435: URL: https://github.com/apache/pinot/issues/12435 Seeing error during loading `Cluster Manager` page. The page tries to send a request to backend to fetch Zookeeper content with path: `//LIVEINSTANCES`.

(pinot) branch master updated: cache ssl contexts and reuse them (#12404)

2024-02-16 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 24f48e3963 cache ssl contexts and reuse them

Re: [PR] cache ssl contexts and reuse them [pinot]

2024-02-16 Thread via GitHub
xiangfu0 merged PR #12404: URL: https://github.com/apache/pinot/pull/12404 -- 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: