Re: [I] [Helm] Update Zookeeper Helm Chart to the latest version [pinot]

2024-05-06 Thread via GitHub
Jackie-Jiang closed issue #13079: [Helm] Update Zookeeper Helm Chart to the latest version URL: https://github.com/apache/pinot/issues/13079 -- 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 speci

(pinot) branch master updated (9bdae816e2 -> 070e3dbd81)

2024-05-06 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 9bdae816e2 Bump com.google.cloud:libraries-bom from 26.37.0 to 26.38.0 (#13045) add 070e3dbd81 [HELM]: zookeeper

Re: [PR] [HELM]: zookeeper chart upgrade to version 13.2.0 [pinot]

2024-05-06 Thread via GitHub
Jackie-Jiang merged PR #13083: URL: https://github.com/apache/pinot/pull/13083 -- 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/com.google.cloud-libraries-bom-26.38.0 deleted (was 27b920e85c)

2024-05-06 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/com.google.cloud-libraries-bom-26.38.0 in repository https://gitbox.apache.org/repos/asf/pinot.git was 27b920e85c Bump com.google.cloud:libraries-bom from 26.37.0 to 26.38.

(pinot) branch master updated (b4dfd04c4d -> 9bdae816e2)

2024-05-06 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 b4dfd04c4d [Minor] Small refactor of raw index creator constructor to be more clear (#13093) add 9bdae816e2 Bump

Re: [PR] Bump com.google.cloud:libraries-bom from 26.37.0 to 26.38.0 [pinot]

2024-05-06 Thread via GitHub
Jackie-Jiang merged PR #13045: URL: https://github.com/apache/pinot/pull/13045 -- 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/com.adobe.testing-s3mock-testcontainers-3.7.3 deleted (was 05e163b3d7)

2024-05-06 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.adobe.testing-s3mock-testcontainers-3.7.3 in repository https://gitbox.apache.org/repos/asf/pinot.git was 05e163b3d7 Bump com.adobe.testing:s3mock-testcontainers fr

Re: [PR] Bump com.adobe.testing:s3mock-testcontainers from 2.17.0 to 3.7.3 [pinot]

2024-05-06 Thread via GitHub
dependabot[bot] commented on PR #13090: URL: https://github.com/apache/pinot/pull/13090#issuecomment-2097522508 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let

Re: [PR] Bump com.adobe.testing:s3mock-testcontainers from 2.17.0 to 3.7.3 [pinot]

2024-05-06 Thread via GitHub
Jackie-Jiang closed pull request #13090: Bump com.adobe.testing:s3mock-testcontainers from 2.17.0 to 3.7.3 URL: https://github.com/apache/pinot/pull/13090 -- 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

Re: [PR] [Minor] Small refactor of raw index creator constructor to be more clear [pinot]

2024-05-06 Thread via GitHub
Jackie-Jiang merged PR #13093: URL: https://github.com/apache/pinot/pull/13093 -- 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 (e07b576797 -> b4dfd04c4d)

2024-05-06 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 e07b576797 Handle shaded classes in all methods of kafka factory (#13087) add b4dfd04c4d [Minor] Small refactor o

Re: [PR] bugfix: fix flakey test, avoid early finalization [pinot]

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

Re: [PR] bugfix: use consumerDir during lucene realtime segment conversion [pinot]

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

Re: [PR] bugfix: use consumerDir during lucene realtime segment conversion [pinot]

2024-05-06 Thread via GitHub
itschrispeck commented on code in PR #13094: URL: https://github.com/apache/pinot/pull/13094#discussion_r1591820799 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/realtime/impl/invertedindex/RealtimeLuceneTextIndex.java: ## @@ -151,9 +151,9 @@ public Mutable

Re: [I] Fix Lucene text index realtime to offline conversion [pinot]

2024-05-06 Thread via GitHub
itschrispeck commented on issue #13039: URL: https://github.com/apache/pinot/issues/13039#issuecomment-2097454638 Hey @stationeros - sorry, but I felt a bit responsible for introducing this bug and have raised a fix to unblock others. -- This is an automated message from the Apache Git S

[PR] Consumers dir fix [pinot]

2024-05-06 Thread via GitHub
itschrispeck opened a new pull request, #13094: URL: https://github.com/apache/pinot/pull/13094 Addresses https://github.com/apache/pinot/issues/13039. Also close `Analyzer` objects, and clarify some exception/log. Testing: Unit tests cover, also sanity tested via quickstart w/ `pin

Re: [PR] dynamic chunk sizing for v4 raw forward index [pinot]

2024-05-06 Thread via GitHub
itschrispeck commented on code in PR #12945: URL: https://github.com/apache/pinot/pull/12945#discussion_r1591797449 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/creator/impl/fwd/SingleValueVarByteRawIndexCreator.java: ## @@ -70,23 +68,32 @@ public

Re: [PR] [Minor] Small refactor of raw index creator constructor to be more clear [pinot]

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

Re: [PR] dynamic chunk sizing for v4 raw forward index [pinot]

2024-05-06 Thread via GitHub
Jackie-Jiang commented on code in PR #12945: URL: https://github.com/apache/pinot/pull/12945#discussion_r1591728925 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/creator/impl/fwd/SingleValueVarByteRawIndexCreator.java: ## @@ -70,23 +68,32 @@ public

Re: [PR] dynamic chunk sizing for v4 raw forward index [pinot]

2024-05-06 Thread via GitHub
klsince commented on code in PR #12945: URL: https://github.com/apache/pinot/pull/12945#discussion_r1591624294 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/creator/impl/fwd/SingleValueVarByteRawIndexCreator.java: ## @@ -70,23 +68,32 @@ public Singl

Re: [PR] dynamic chunk sizing for v4 raw forward index [pinot]

2024-05-06 Thread via GitHub
klsince commented on code in PR #12945: URL: https://github.com/apache/pinot/pull/12945#discussion_r1591624294 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/creator/impl/fwd/SingleValueVarByteRawIndexCreator.java: ## @@ -70,23 +68,32 @@ public Singl

Re: [I] Dependency issue detected in AdminConsoleIntegrationTest [pinot]

2024-05-06 Thread via GitHub
abhioncbr commented on issue #13047: URL: https://github.com/apache/pinot/issues/13047#issuecomment-2097015230 I am looking into this. Thanks -- 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 s

Re: [PR] Bugfix: Validate minionInstanceTag during task-generation [pinot]

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

[PR] Bugfix: Validate minionInstanceTag during task-generation [pinot]

2024-05-06 Thread via GitHub
tibrewalpratik17 opened a new pull request, #13092: URL: https://github.com/apache/pinot/pull/13092 label: `bugfix` This patch validates whether the configured minionInstanceTag actually exists in the list of instance-tags or not. Before this, if a wrong instanceTag was configured

Re: [PR] Introduce retries while creating stream message decoder for more robustness [pinot]

2024-05-06 Thread via GitHub
tibrewalpratik17 commented on PR #13036: URL: https://github.com/apache/pinot/pull/13036#issuecomment-2096537166 hey @swaminathanmanish can you help with review? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] Add support for creating raw derived columns during segment reload [pinot]

2024-05-06 Thread via GitHub
yashmayya commented on code in PR #13037: URL: https://github.com/apache/pinot/pull/13037#discussion_r1590974745 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/loader/defaultcolumn/BaseDefaultColumnHandler.java: ## @@ -757,57 +759,181 @@ privat

Re: [PR] Optimise Sorting in SegmentProcessorFramework [WIP] [pinot]

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

[PR] Optimise Sorting in SegmentProcessorFramework [WIP] [pinot]

2024-05-06 Thread via GitHub
aishikbh opened a new pull request, #13091: URL: https://github.com/apache/pinot/pull/13091 Load only the sort columns in memory and call compare on the loaded columns instead of calling compare on the whole mmapped datafile -- This is an automated message from the Apache Git Service. To

Re: [PR] Add support for creating raw derived columns during segment reload [pinot]

2024-05-06 Thread via GitHub
yashmayya commented on code in PR #13037: URL: https://github.com/apache/pinot/pull/13037#discussion_r1590974745 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/loader/defaultcolumn/BaseDefaultColumnHandler.java: ## @@ -757,57 +759,181 @@ privat

(pinot) branch master updated: Handle shaded classes in all methods of kafka factory (#13087)

2024-05-06 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 e07b576797 Handle shaded classes in all methods o

Re: [PR] Handle shaded classes in all methods of kafka factory [pinot]

2024-05-06 Thread via GitHub
gortiz merged PR #13087: URL: https://github.com/apache/pinot/pull/13087 -- 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

Re: [PR] Handle shaded classes in all methods of kafka factory [pinot]

2024-05-06 Thread via GitHub
gortiz commented on code in PR #13087: URL: https://github.com/apache/pinot/pull/13087#discussion_r1590946437 ## pinot-plugins/pinot-stream-ingestion/pinot-kafka-2.0/src/main/java/org/apache/pinot/plugin/stream/kafka20/KafkaConfigBackwardCompatibleUtils.java: ## @@ -35,11 +35,13

Re: [I] Exceptions when opening Pinot Swagger UI [pinot]

2024-05-06 Thread via GitHub
abhioncbr commented on issue #5306: URL: https://github.com/apache/pinot/issues/5306#issuecomment-2095819204 > This is similar, but not sure if we want to take this approach: https://stackoverflow.com/questions/49875277/swagger-hk2-service-reification-failed I tried this option, but i

Re: [PR] Handle shaded classes in all methods of kafka factory [pinot]

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

(pinot) branch dependabot/maven/com.fasterxml.jackson-jackson-bom-2.17.1 created (now 57298a282f)

2024-05-06 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.fasterxml.jackson-jackson-bom-2.17.1 in repository https://gitbox.apache.org/repos/asf/pinot.git at 57298a282f Bump com.fasterxml.jackson:jackson-bom from 2.12.7.2

[PR] Bump com.adobe.testing:s3mock-testcontainers from 2.17.0 to 3.7.3 [pinot]

2024-05-06 Thread via GitHub
dependabot[bot] opened a new pull request, #13090: URL: https://github.com/apache/pinot/pull/13090 Bumps com.adobe.testing:s3mock-testcontainers from 2.17.0 to 3.7.3. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-n

[PR] Bump com.fasterxml.jackson:jackson-bom from 2.12.7.20221012 to 2.17.1 [pinot]

2024-05-06 Thread via GitHub
dependabot[bot] opened a new pull request, #13089: URL: https://github.com/apache/pinot/pull/13089 Bumps [com.fasterxml.jackson:jackson-bom](https://github.com/FasterXML/jackson-bom) from 2.12.7.20221012 to 2.17.1. Commits https://github.com/FasterXML/jackson-bom/commit/50ab40

(pinot) branch dependabot/maven/aws.sdk.version-2.25.45 created (now 9a83e3695f)

2024-05-06 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/aws.sdk.version-2.25.45 in repository https://gitbox.apache.org/repos/asf/pinot.git at 9a83e3695f Bump aws.sdk.version from 2.25.44 to 2.25.45 No new revisions were ad

[PR] Bump aws.sdk.version from 2.25.44 to 2.25.45 [pinot]

2024-05-06 Thread via GitHub
dependabot[bot] opened a new pull request, #13088: URL: https://github.com/apache/pinot/pull/13088 Bumps `aws.sdk.version` from 2.25.44 to 2.25.45. Updates `software.amazon.awssdk:bom` from 2.25.44 to 2.25.45 Updates `software.amazon.awssdk:apache-client` from 2.25.44 to 2.25.45

(pinot) branch dependabot/maven/com.adobe.testing-s3mock-testcontainers-3.7.3 created (now 05e163b3d7)

2024-05-06 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.adobe.testing-s3mock-testcontainers-3.7.3 in repository https://gitbox.apache.org/repos/asf/pinot.git at 05e163b3d7 Bump com.adobe.testing:s3mock-testcontainers fr

[PR] Handle shaded classes in all methods of kafka factory [pinot]

2024-05-06 Thread via GitHub
KKcorps opened a new pull request, #13087: URL: https://github.com/apache/pinot/pull/13087 (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-ma