Re: [PR] Part-2: Add Combine and Segment Level Operators for Time Series [pinot]

2024-09-16 Thread via GitHub
ankitsultana commented on code in PR #13999: URL: https://github.com/apache/pinot/pull/13999#discussion_r1762432674 ## pinot-core/src/main/java/org/apache/pinot/core/operator/timeseries/TimeSeriesAggregationOperator.java: ## @@ -0,0 +1,251 @@ +/** + * Licensed to the Apache Soft

Re: [PR] Part-2: Add Combine and Segment Level Operators for Time Series [pinot]

2024-09-16 Thread via GitHub
ankitsultana commented on code in PR #13999: URL: https://github.com/apache/pinot/pull/13999#discussion_r1762433863 ## pinot-core/src/main/java/org/apache/pinot/core/query/request/ServerQueryRequest.java: ## @@ -107,6 +107,28 @@ public ServerQueryRequest(Server.ServerRequest se

Re: [PR] Part-2: Add Combine and Segment Level Operators for Time Series [pinot]

2024-09-16 Thread via GitHub
ankitsultana commented on code in PR #13999: URL: https://github.com/apache/pinot/pull/13999#discussion_r1762430749 ## pinot-core/src/main/java/org/apache/pinot/core/operator/timeseries/TimeSeriesAggregationOperator.java: ## @@ -0,0 +1,251 @@ +/** + * Licensed to the Apache Soft

Re: [PR] Part-2: Add Combine and Segment Level Operators for Time Series [pinot]

2024-09-16 Thread via GitHub
ankitsultana commented on code in PR #13999: URL: https://github.com/apache/pinot/pull/13999#discussion_r1762429676 ## pinot-core/src/main/java/org/apache/pinot/core/operator/timeseries/TimeSeriesAggregationOperator.java: ## @@ -0,0 +1,251 @@ +/** + * Licensed to the Apache Soft

Re: [PR] Part-2: Add Combine and Segment Level Operators for Time Series [pinot]

2024-09-16 Thread via GitHub
ankitsultana commented on code in PR #13999: URL: https://github.com/apache/pinot/pull/13999#discussion_r1762387492 ## pinot-core/src/main/java/org/apache/pinot/core/operator/combine/merger/TimeSeriesAggResultsBlockMerger.java: ## @@ -0,0 +1,66 @@ +/** + * Licensed to the Apache

Re: [PR] Part-2: Add Combine and Segment Level Operators for Time Series [pinot]

2024-09-16 Thread via GitHub
ankitsultana commented on code in PR #13999: URL: https://github.com/apache/pinot/pull/13999#discussion_r1762382031 ## pinot-common/src/main/java/org/apache/pinot/common/request/context/TimeSeriesContext.java: ## @@ -0,0 +1,73 @@ +/** + * Licensed to the Apache Software Foundati

Re: [I] Funnel Count Query Failing with predicate filters. [pinot]

2024-09-16 Thread via GitHub
Jayesh-Asrani commented on issue #13593: URL: https://github.com/apache/pinot/issues/13593#issuecomment-2354559376 This seems to be resolved in the latest versions. We can close this -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

Re: [I] Funnel Count Query Failing with predicate filters. [pinot]

2024-09-16 Thread via GitHub
Jayesh-Asrani closed issue #13593: Funnel Count Query Failing with predicate filters. URL: https://github.com/apache/pinot/issues/13593 -- 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 c

Re: [PR] Storage Quota imposition on Realtime tables [pinot]

2024-09-16 Thread via GitHub
shounakmk219 commented on code in PR #13584: URL: https://github.com/apache/pinot/pull/13584#discussion_r1762202266 ## pinot-controller/src/main/java/org/apache/pinot/controller/validation/RealtimeSegmentValidationManager.java: ## @@ -108,8 +112,45 @@ protected void processTable

Re: [PR] track newly added segments for upsert tables for a more complete upsert data view [pinot]

2024-09-16 Thread via GitHub
klsince commented on code in PR #13992: URL: https://github.com/apache/pinot/pull/13992#discussion_r1762182421 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java: ## @@ -1200,12 +1219,45 @@ public void trackSegmentF

Re: [PR] track newly added segments for upsert tables for a more complete upsert data view [pinot]

2024-09-16 Thread via GitHub
Jackie-Jiang commented on code in PR #13992: URL: https://github.com/apache/pinot/pull/13992#discussion_r1762165645 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java: ## @@ -1200,12 +1219,45 @@ public void trackSeg

Re: [I] [Backward incompatible issues in 1.1] segmentAssignmentStrategy can be NULL after it is deprecated in SegmentValidationAndRetentionConfig [pinot]

2024-09-16 Thread via GitHub
MeihanLi closed issue #13659: [Backward incompatible issues in 1.1] segmentAssignmentStrategy can be NULL after it is deprecated in SegmentValidationAndRetentionConfig URL: https://github.com/apache/pinot/issues/13659 -- This is an automated message from the Apache Git Service. To respond to

Re: [I] [Backward incompatible issues in 1.1] segmentAssignmentStrategy can be NULL after it is deprecated in SegmentValidationAndRetentionConfig [pinot]

2024-09-16 Thread via GitHub
MeihanLi commented on issue #13659: URL: https://github.com/apache/pinot/issues/13659#issuecomment-2354339321 yes, we control the table config creation so it's a big change for us. As long as leaving it as NULL works, we are fine with it. Thanks @Jackie-Jiang for taking a look at this issue

Re: [I] Apache Pinot vulnerabilities issues [pinot]

2024-09-16 Thread via GitHub
abhioncbr commented on issue #13461: URL: https://github.com/apache/pinot/issues/13461#issuecomment-2354318105 The following command shows, below vulnearbilties ``` docker scout cves apachepinot/pinot:1.2.0-21-openjdk 127 vulnerabilities found in 36 packages UNSPECIFIED 9

Re: [I] Apache Pinot vulnerabilities issues [pinot]

2024-09-16 Thread via GitHub
mayankshriv commented on issue #13461: URL: https://github.com/apache/pinot/issues/13461#issuecomment-2354186899 @vpriyam Could you please share what tool was used to flag these CVEs? Also, I spot checked a few, I am unclear how are they related to Pinot. For example, the first one is about

Re: [PR] Enforce removeSegment flow with _enableDeletedKeysCompactionConsistency [pinot]

2024-09-16 Thread via GitHub
klsince commented on code in PR #13914: URL: https://github.com/apache/pinot/pull/13914#discussion_r1761999187 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/ConcurrentMapPartitionUpsertMetadataManagerForConsistentDeletes.java: ## @@ -81,6 +85,12 @@ p

Re: [PR] Enforce removeSegment flow with _enableDeletedKeysCompactionConsistency [pinot]

2024-09-16 Thread via GitHub
tibrewalpratik17 commented on code in PR #13914: URL: https://github.com/apache/pinot/pull/13914#discussion_r1761969294 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java: ## @@ -544,9 +544,7 @@ public void addSegme

Re: [PR] track newly added segments for upsert tables for a more complete upsert data view [pinot]

2024-09-16 Thread via GitHub
klsince commented on code in PR #13992: URL: https://github.com/apache/pinot/pull/13992#discussion_r1761893831 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java: ## @@ -153,11 +164,17 @@ protected BasePartitionUpse

Re: [PR] Storage Quota imposition on Realtime tables [pinot]

2024-09-16 Thread via GitHub
mcvsubbu commented on code in PR #13584: URL: https://github.com/apache/pinot/pull/13584#discussion_r1761941816 ## pinot-controller/src/main/java/org/apache/pinot/controller/validation/RealtimeSegmentValidationManager.java: ## @@ -108,8 +112,45 @@ protected void processTable(Str

Re: [PR] [WIP] [Feature] Enable the capability to specify zstd and lz4 segment compression via config [pinot]

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

Re: [PR] track newly added segments for upsert tables for a more complete upsert data view [pinot]

2024-09-16 Thread via GitHub
klsince commented on code in PR #13992: URL: https://github.com/apache/pinot/pull/13992#discussion_r1761893831 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java: ## @@ -153,11 +164,17 @@ protected BasePartitionUpse

Re: [PR] track newly added segments for upsert tables for a more complete upsert data view [pinot]

2024-09-16 Thread via GitHub
klsince commented on code in PR #13992: URL: https://github.com/apache/pinot/pull/13992#discussion_r1761893831 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java: ## @@ -153,11 +164,17 @@ protected BasePartitionUpse

Re: [PR] track newly added segments for upsert tables for a more complete upsert data view [pinot]

2024-09-16 Thread via GitHub
klsince commented on code in PR #13992: URL: https://github.com/apache/pinot/pull/13992#discussion_r1761853000 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java: ## @@ -1200,12 +1219,45 @@ public void trackSegmentF

[PR] [WIP] [Feature] Enable the capability to specify zstd and lz4 segment compression via config [pinot]

2024-09-16 Thread via GitHub
jackluo923 opened a new pull request, #14008: URL: https://github.com/apache/pinot/pull/14008 # Summary This PR builds on #13782 by adding support for specifying segment compression using Zstandard and LZ4 via configuration. By default, segments are compressed with GZIP. However, users c

Re: [PR] Enforce removeSegment flow with _enableDeletedKeysCompactionConsistency [pinot]

2024-09-16 Thread via GitHub
klsince commented on code in PR #13914: URL: https://github.com/apache/pinot/pull/13914#discussion_r1761757917 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java: ## @@ -544,9 +544,7 @@ public void addSegment(Immuta

Re: [PR] Add offset based lag metrics [pinot]

2024-09-16 Thread via GitHub
jadami10 commented on PR #13298: URL: https://github.com/apache/pinot/pull/13298#issuecomment-2353694120 the other big issue I see is if we fail to request the latest offset, we stop publishing both the time based and offset based ingestion lag. -- This is an automated message from the Ap

Re: [PR] Part-2: Add Combine and Segment Level Operators for Time Series [pinot]

2024-09-16 Thread via GitHub
raghavyadav01 commented on code in PR #13999: URL: https://github.com/apache/pinot/pull/13999#discussion_r1761553293 ## pinot-common/src/main/java/org/apache/pinot/common/request/context/TimeSeriesContext.java: ## @@ -0,0 +1,73 @@ +/** + * Licensed to the Apache Software Foundat

Re: [PR] track newly added segments for upsert tables for a more complete upsert data view [pinot]

2024-09-16 Thread via GitHub
Jackie-Jiang commented on code in PR #13992: URL: https://github.com/apache/pinot/pull/13992#discussion_r1761577304 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java: ## @@ -1200,12 +1219,45 @@ public void trackSeg

Re: [PR] Add offset based lag metrics [pinot]

2024-09-16 Thread via GitHub
jadami10 commented on PR #13298: URL: https://github.com/apache/pinot/pull/13298#issuecomment-2353497319 we're upgrading to 1.2 now, and this increased our kafka offset requests in our QA environment by 3 orders of magnitude. I think at minimum we need a way to turn this off (internally we'

Re: [PR] [WIP]Dynamic evaluation of GroupBy Initial Capacity [pinot]

2024-09-16 Thread via GitHub
vvivekiyer commented on code in PR #14001: URL: https://github.com/apache/pinot/pull/14001#discussion_r1761509972 ## pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/groupby/DefaultGroupByExecutor.java: ## @@ -183,4 +194,26 @@ public GroupKeyGenerator getGroupKey

(pinot) branch master updated: Bump software.amazon.awssdk:bom from 2.28.0 to 2.28.1 (#14007)

2024-09-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 b47fae9a3c Bump software.amazon.awssdk:bom from 2

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.28.1 deleted (was 7bd67ba1b1)

2024-09-16 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.28.1 in repository https://gitbox.apache.org/repos/asf/pinot.git was 7bd67ba1b1 Bump software.amazon.awssdk:bom from 2.28.0 to 2.28.1 The revi

Re: [PR] Bump software.amazon.awssdk:bom from 2.28.0 to 2.28.1 [pinot]

2024-09-16 Thread via GitHub
Jackie-Jiang merged PR #14007: URL: https://github.com/apache/pinot/pull/14007 -- 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 dependabot/maven/joda-time-joda-time-2.13.0 deleted (was 3dca83089d)

2024-09-16 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/joda-time-joda-time-2.13.0 in repository https://gitbox.apache.org/repos/asf/pinot.git was 3dca83089d Bump joda-time:joda-time from 2.12.7 to 2.13.0 The revisions that were

Re: [PR] Bump joda-time:joda-time from 2.12.7 to 2.13.0 [pinot]

2024-09-16 Thread via GitHub
Jackie-Jiang merged PR #14006: URL: https://github.com/apache/pinot/pull/14006 -- 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: Bump joda-time:joda-time from 2.12.7 to 2.13.0 (#14006)

2024-09-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 f25b3f11d1 Bump joda-time:joda-time from 2.12.7 t

(pinot) branch dependabot/maven/jna.version-5.15.0 deleted (was 6f713968d4)

2024-09-16 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/jna.version-5.15.0 in repository https://gitbox.apache.org/repos/asf/pinot.git was 6f713968d4 Bump jna.version from 5.14.0 to 5.15.0 The revisions that were on this branch

(pinot) branch master updated: Bump jna.version from 5.14.0 to 5.15.0 (#14004)

2024-09-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 074fffcbf4 Bump jna.version from 5.14.0 to 5.15.0

Re: [PR] Bump jna.version from 5.14.0 to 5.15.0 [pinot]

2024-09-16 Thread via GitHub
Jackie-Jiang merged PR #14004: URL: https://github.com/apache/pinot/pull/14004 -- 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: [I] SegmentTarPushJobRunner NullPointerException import csv [pinot]

2024-09-16 Thread via GitHub
xiangfu0 commented on issue #11940: URL: https://github.com/apache/pinot/issues/11940#issuecomment-2352770691 For M1, can you try to explicitly use the image with arm64 tag, e.g. [apachepinot/pinot:1.2.0-21-openjdk-linux-arm64](https://hub.docker.com/layers/apachepinot/pinot/1.2.0-21-openjdk

(pinot) branch master updated (4a26fba3c3 -> 3df697cb48)

2024-09-16 Thread xiangfu
This is an automated email from the ASF dual-hosted git repository. xiangfu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from 4a26fba3c3 minor: changes as a follow up PR (#14000) add 3df697cb48 Kafka3 support (#13891) No new revisions wer

Re: [PR] Kafka3 support [pinot]

2024-09-16 Thread via GitHub
xiangfu0 merged PR #13891: URL: https://github.com/apache/pinot/pull/13891 -- 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 dependabot/maven/software.amazon.awssdk-bom-2.28.1 created (now 7bd67ba1b1)

2024-09-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/software.amazon.awssdk-bom-2.28.1 in repository https://gitbox.apache.org/repos/asf/pinot.git at 7bd67ba1b1 Bump software.amazon.awssdk:bom from 2.28.0 to 2.28.1 No n

[PR] Bump software.amazon.awssdk:bom from 2.28.0 to 2.28.1 [pinot]

2024-09-16 Thread via GitHub
dependabot[bot] opened a new pull request, #14007: URL: https://github.com/apache/pinot/pull/14007 Bumps software.amazon.awssdk:bom from 2.28.0 to 2.28.1. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=software

(pinot) branch dependabot/maven/joda-time-joda-time-2.13.0 created (now 3dca83089d)

2024-09-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/joda-time-joda-time-2.13.0 in repository https://gitbox.apache.org/repos/asf/pinot.git at 3dca83089d Bump joda-time:joda-time from 2.12.7 to 2.13.0 No new revisions we

[PR] Bump joda-time:joda-time from 2.12.7 to 2.13.0 [pinot]

2024-09-16 Thread via GitHub
dependabot[bot] opened a new pull request, #14006: URL: https://github.com/apache/pinot/pull/14006 Bumps [joda-time:joda-time](https://github.com/JodaOrg/joda-time) from 2.12.7 to 2.13.0. Release notes Sourced from https://github.com/JodaOrg/joda-time/releases";>joda-time:joda-time

(pinot) branch dependabot/maven/org.apache.datasketches-datasketches-java-6.1.0 created (now 8a3af13487)

2024-09-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/org.apache.datasketches-datasketches-java-6.1.0 in repository https://gitbox.apache.org/repos/asf/pinot.git at 8a3af13487 Bump org.apache.datasketches:datasketches-jav

[PR] Bump org.apache.datasketches:datasketches-java from 6.0.0 to 6.1.0 [pinot]

2024-09-16 Thread via GitHub
dependabot[bot] opened a new pull request, #14005: URL: https://github.com/apache/pinot/pull/14005 Bumps org.apache.datasketches:datasketches-java from 6.0.0 to 6.1.0. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-

[PR] Bump jna.version from 5.14.0 to 5.15.0 [pinot]

2024-09-16 Thread via GitHub
dependabot[bot] opened a new pull request, #14004: URL: https://github.com/apache/pinot/pull/14004 Bumps `jna.version` from 5.14.0 to 5.15.0. Updates `net.java.dev.jna:jna` from 5.14.0 to 5.15.0 Changelog Sourced from https://github.com/java-native-access/jna/blob/master/CHANGES.

(pinot) branch dependabot/maven/jna.version-5.15.0 created (now 6f713968d4)

2024-09-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/jna.version-5.15.0 in repository https://gitbox.apache.org/repos/asf/pinot.git at 6f713968d4 Bump jna.version from 5.14.0 to 5.15.0 No new revisions were added by this

Re: [PR] Increase unit test coverage for AbstractMetrics [pinot]

2024-09-16 Thread via GitHub
tibrewalpratik17 commented on PR #13991: URL: https://github.com/apache/pinot/pull/13991#issuecomment-2352628957 Thanks @spanasch for this! Left few nit comments, overall LGTM! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Increase unit test coverage for AbstractMetrics [pinot]

2024-09-16 Thread via GitHub
tibrewalpratik17 commented on code in PR #13991: URL: https://github.com/apache/pinot/pull/13991#discussion_r1760953848 ## pinot-common/src/test/java/org/apache/pinot/common/metrics/AbstractMetricsTest.java: ## @@ -108,4 +112,154 @@ public void testMultipleGauges() { contro

Re: [PR] Increase unit test coverage for AbstractMetrics [pinot]

2024-09-16 Thread via GitHub
tibrewalpratik17 commented on code in PR #13991: URL: https://github.com/apache/pinot/pull/13991#discussion_r1760951898 ## pinot-common/src/test/java/org/apache/pinot/common/metrics/AbstractMetricsTest.java: ## @@ -108,4 +112,154 @@ public void testMultipleGauges() { contro

Re: [PR] Increase unit test coverage for AbstractMetrics [pinot]

2024-09-16 Thread via GitHub
tibrewalpratik17 commented on code in PR #13991: URL: https://github.com/apache/pinot/pull/13991#discussion_r1760939827 ## pinot-common/src/test/java/org/apache/pinot/common/metrics/AbstractMetricsTest.java: ## @@ -108,4 +112,154 @@ public void testMultipleGauges() { contro

Re: [PR] Increase unit test coverage for AbstractMetrics [pinot]

2024-09-16 Thread via GitHub
tibrewalpratik17 commented on code in PR #13991: URL: https://github.com/apache/pinot/pull/13991#discussion_r1760939827 ## pinot-common/src/test/java/org/apache/pinot/common/metrics/AbstractMetricsTest.java: ## @@ -108,4 +112,154 @@ public void testMultipleGauges() { contro

Re: [PR] Increase unit test coverage for AbstractMetrics [pinot]

2024-09-16 Thread via GitHub
tibrewalpratik17 commented on code in PR #13991: URL: https://github.com/apache/pinot/pull/13991#discussion_r1760939827 ## pinot-common/src/test/java/org/apache/pinot/common/metrics/AbstractMetricsTest.java: ## @@ -108,4 +112,154 @@ public void testMultipleGauges() { contro

Re: [PR] Increase unit test coverage for AbstractMetrics [pinot]

2024-09-16 Thread via GitHub
tibrewalpratik17 commented on code in PR #13991: URL: https://github.com/apache/pinot/pull/13991#discussion_r1760938266 ## pinot-common/src/test/java/org/apache/pinot/common/metrics/AbstractMetricsTest.java: ## @@ -108,4 +112,154 @@ public void testMultipleGauges() { contro

Re: [PR] Less logging during tests [pinot]

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

Re: [PR] Don't replace environment variables and system properties in get table configs REST API [pinot]

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