[PR] Upgrade jettison version to fix CVE [pinot]

2024-03-05 Thread via GitHub
xiangfu0 opened a new pull request, #12567: URL: https://github.com/apache/pinot/pull/12567 Upgrade jettison version to 1.5.4 to fix: CVE-2023-1436 CVE-2022-45693 CVE-2022-45685 CVE-2022-40150 CVE-2022-40149 -- This is an automated message from the Apache Git Service. To

(pinot) branch master updated: fixing CVE critical issues by resolving kerby/jline and wildfly libraries (#12566)

2024-03-05 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 2e84561985 fixing CVE critical issues by resolvi

Re: [PR] fixing CVE critical issues by resolving kerby/jline and wildfly libraries [pinot]

2024-03-05 Thread via GitHub
xiangfu0 merged PR #12566: URL: https://github.com/apache/pinot/pull/12566 -- 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 issue with startree index metadata loading for columns with '__' in name (#12554)

2024-03-05 Thread saurabhd336
This is an automated email from the ASF dual-hosted git repository. saurabhd336 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 083ab4e671 Fix issue with startree index met

Re: [PR] Fix issue with startree index metadata loading for columns with '__' in name [pinot]

2024-03-05 Thread via GitHub
saurabhd336 merged PR #12554: URL: https://github.com/apache/pinot/pull/12554 -- 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] Rebalancing table with no segments and "reassign instances" returns IN_PROGRESS but doesn't track the job_id or NullPointerException [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on issue #12316: URL: https://github.com/apache/pinot/issues/12316#issuecomment-1980247916 This is unexpected. When there is no segment, when invoking `TableRebalancer.doRebalance()`, `segmentAssignmentUnchanged` should be `true` because both current and target assign

Re: [PR] bugfix: do not move src ByteBuffer position for LZ4 length prefixed decompress [pinot]

2024-03-05 Thread via GitHub
itschrispeck commented on PR #12539: URL: https://github.com/apache/pinot/pull/12539#issuecomment-1980247003 > Trying to understand the problem better here. Why would it cause problem when the source position is moved? It is allowed for `decompress()` to move the position of the compressed

Re: [I] Minion task for hybrid table erroring about a retry attempt exhaustion [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on issue #12547: URL: https://github.com/apache/pinot/issues/12547#issuecomment-1980238251 Can you also check the WARN log from minion? @snleee @swaminathanmanish Please help take a look -- This is an automated message from the Apache Git Service. To respond t

Re: [PR] Fix issue with startree index metadata loading for columns with '__' in name [pinot]

2024-03-05 Thread via GitHub
gortiz commented on code in PR #12554: URL: https://github.com/apache/pinot/pull/12554#discussion_r1513946123 ## pinot-core/src/test/java/org/apache/pinot/core/startree/v2/BaseStarTreeV2Test.java: ## @@ -91,32 +91,35 @@ abstract class BaseStarTreeV2Test { private static fin

Re: [PR] fixing CVE critical issues by resolving kerby/jline and wildfly libraries [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on code in PR #12566: URL: https://github.com/apache/pinot/pull/12566#discussion_r1513932252 ## pom.xml: ## @@ -857,6 +860,10 @@ commons-logging commons-logging + Review Comment: Figured that netty is the

Re: [I] Seems text_match() function doesn't support search with '#' character [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on issue #12262: URL: https://github.com/apache/pinot/issues/12262#issuecomment-1980207046 I saw this similar [issue](https://discuss.elastic.co/t/lucene-query-for-string-containing-hash-symbol/316429) posted for Elastic search, but no response.. -- This is an auto

Re: [I] [Flaky test] CaseTransformFunctionTest. testCaseTransformFunctionWithIntResults() [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on issue #12367: URL: https://github.com/apache/pinot/issues/12367#issuecomment-1980200987 When casting `0.1f` to `double`, the result is actually `0.1000149011612`, which is not equal to `0.1d`. We might hit this scenario because the literal is parsed as double i

(pinot) branch master updated (87c1522bf7 -> 9f5b126a9a)

2024-03-05 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 87c1522bf7 use https (#12551) add 9f5b126a9a Update kotlin and okio dependency version (#12565) No new revisions

Re: [PR] Update kotlin and okio dependency version [pinot]

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

Re: [PR] Fix issue with startree index metadata loading for columns with '__' in name [pinot]

2024-03-05 Thread via GitHub
saurabhd336 commented on code in PR #12554: URL: https://github.com/apache/pinot/pull/12554#discussion_r1513910331 ## pinot-core/src/test/java/org/apache/pinot/core/startree/v2/BaseStarTreeV2Test.java: ## @@ -91,32 +91,35 @@ abstract class BaseStarTreeV2Test { private stati

Re: [PR] [experiment] Deterministic Hash Function for Upsert Type-4 UUID Primary Keys [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on code in PR #12538: URL: https://github.com/apache/pinot/pull/12538#discussion_r1513907963 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/HashUtils.java: ## @@ -36,6 +40,35 @@ public static byte[] hashMD5(byte[] bytes) { r

Re: [PR] fixing CVE critical issues by resolving kerby/jline and wildfly libraries [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on code in PR #12566: URL: https://github.com/apache/pinot/pull/12566#discussion_r1513904333 ## pom.xml: ## @@ -857,6 +860,10 @@ commons-logging commons-logging + Review Comment: I don't follow. Since we

Re: [PR] Fix issue with startree index metadata loading for columns with '__' in name [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on code in PR #12554: URL: https://github.com/apache/pinot/pull/12554#discussion_r1513901059 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/startree/v2/store/StarTreeIndexMapUtils.java: ## @@ -198,9 +199,9 @@ public static List> loadF

Re: [PR] fixing CVE critical issues by resolving kerby/jline and wildfly libraries [pinot]

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

Re: [PR] Update kotlin and okio dependency version [pinot]

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

Re: [PR] fixing CVE critical issues by resolving kerby/jline and wildfly libraries [pinot]

2024-03-05 Thread via GitHub
xiangfu0 commented on code in PR #12566: URL: https://github.com/apache/pinot/pull/12566#discussion_r1513882760 ## pom.xml: ## @@ -857,6 +860,10 @@ commons-logging commons-logging + Review Comment: We are using `io.netty:netty-

Re: [PR] fixing CVE critical issues by resolving kerby/jline and wildfly libraries [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on code in PR #12566: URL: https://github.com/apache/pinot/pull/12566#discussion_r1513876168 ## pom.xml: ## @@ -857,6 +860,10 @@ commons-logging commons-logging + Review Comment: Do we need to exclude net

Re: [PR] Fix issue with startree index metadata loading for columns with '__' in name [pinot]

2024-03-05 Thread via GitHub
saurabhd336 commented on PR #12554: URL: https://github.com/apache/pinot/pull/12554#issuecomment-1980154856 @Jackie-Jiang I guess going forward that sort of enforcement on column name makes sense, but it may break loading of existing segments which have columns with double underscore in the

(pinot) branch master updated (83b148fbf7 -> 87c1522bf7)

2024-03-05 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 83b148fbf7 Bump com.h2database:h2 from 2.2.220 to 2.2.224 (#12470) add 87c1522bf7 use https (#12551) No new revis

Re: [PR] use https [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang merged PR #12551: URL: https://github.com/apache/pinot/pull/12551 -- 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

[PR] fixing CVE critical issues by resolving kerby/jline and wildfly libraries [pinot]

2024-03-05 Thread via GitHub
xiangfu0 opened a new pull request, #12566: URL: https://github.com/apache/pinot/pull/12566 1. Remove io.netty:netty library 2. Consolidate org.apache.kerby:kerb-core and org.apache.kerby:kerb-simplekdc version to 2.0.3 3. Consolidate org.jline:jline version to 3.22.0 4. Consolidate

(pinot) branch master updated (4803f49c4b -> 83b148fbf7)

2024-03-05 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 4803f49c4b reformat AdminConsoleIntegrationTest.java (#12552) add 83b148fbf7 Bump com.h2database:h2 from 2.2.220 t

(pinot) branch dependabot/maven/com.h2database-h2-2.2.224 deleted (was e051ac984b)

2024-03-05 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/com.h2database-h2-2.2.224 in repository https://gitbox.apache.org/repos/asf/pinot.git was e051ac984b Bump com.h2database:h2 from 2.2.220 to 2.2.224 The revisions that were

Re: [PR] Bump com.h2database:h2 from 2.2.220 to 2.2.224 [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang merged PR #12470: URL: https://github.com/apache/pinot/pull/12470 -- 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 (7f8b5db022 -> 4803f49c4b)

2024-03-05 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 7f8b5db022 Bump nl.jqno.equalsverifier:equalsverifier from 3.14.1 to 3.15.8 (#12553) add 4803f49c4b reformat Admi

Re: [PR] reformat AdminConsoleIntegrationTest.java [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang merged PR #12552: URL: https://github.com/apache/pinot/pull/12552 -- 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 (fec8d76138 -> 7f8b5db022)

2024-03-05 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 fec8d76138 add test cases in RequestUtilsTest (#12557) add 7f8b5db022 Bump nl.jqno.equalsverifier:equalsverifier f

Re: [PR] Bump nl.jqno.equalsverifier:equalsverifier from 3.14.1 to 3.15.8 [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang merged PR #12553: URL: https://github.com/apache/pinot/pull/12553 -- 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/nl.jqno.equalsverifier-equalsverifier-3.15.8 deleted (was 8ca6e96b97)

2024-03-05 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/nl.jqno.equalsverifier-equalsverifier-3.15.8 in repository https://gitbox.apache.org/repos/asf/pinot.git was 8ca6e96b97 Bump nl.jqno.equalsverifier:equalsverifier from 3.14

(pinot) branch master updated: add test cases in RequestUtilsTest (#12557)

2024-03-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 fec8d76138 add test cases in RequestUtilsTest (#1

Re: [PR] add test cases in RequestUtilsTest [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang merged PR #12557: URL: https://github.com/apache/pinot/pull/12557 -- 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: aws sdk 2.25.3 (#12562)

2024-03-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 9205494cc1 aws sdk 2.25.3 (#12562) 9205494cc1 is

Re: [PR] aws sdk 2.25.3 [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang merged PR #12562: URL: https://github.com/apache/pinot/pull/12562 -- 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] Bump com.squareup.okio:okio from 2.10.0 to 3.4.0 [pinot]

2024-03-05 Thread via GitHub
dependabot[bot] commented on PR #12563: URL: https://github.com/apache/pinot/pull/12563#issuecomment-1980122237 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

(pinot) branch dependabot/maven/com.squareup.okio-okio-3.4.0 deleted (was c6d7ba892c)

2024-03-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/com.squareup.okio-okio-3.4.0 in repository https://gitbox.apache.org/repos/asf/pinot.git was c6d7ba892c Bump com.squareup.okio:okio from 2.10.0 to 3.4.0 The revisions

Re: [PR] Bump com.squareup.okio:okio from 2.10.0 to 3.4.0 [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang closed pull request #12563: Bump com.squareup.okio:okio from 2.10.0 to 3.4.0 URL: https://github.com/apache/pinot/pull/12563 -- 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

[PR] Update kotlin and okio dependency version [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang opened a new pull request, #12565: URL: https://github.com/apache/pinot/pull/12565 Also removes the variables for `reload4j` and `jaxb-api` version as they are no longer maintained and we shouldn't need to update them -- This is an automated message from the Apache Git Servic

Re: [PR] Fix issue with startree index metadata loading for columns with '__' in name [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on PR #12554: URL: https://github.com/apache/pinot/pull/12554#issuecomment-1980108642 I believe we chose double underscore as delimiter because it should be forbidden as column name, but somehow we didn't enforce it during schema validation -- This is an automated

Re: [PR] Allow passing database context through `database` http header [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on code in PR #12417: URL: https://github.com/apache/pinot/pull/12417#discussion_r1513822889 ## pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseBrokerRequestHandler.java: ## @@ -320,6 +321,11 @@ protected BrokerResponse handleRequest(

Re: [PR] Fix issue with startree index metadata loading for columns with '__' in name [pinot]

2024-03-05 Thread via GitHub
KKcorps commented on PR #12554: URL: https://github.com/apache/pinot/pull/12554#issuecomment-1980098568 LGTM, @saurabhd336 just also check if these metadata columns from input streams are also not affected https://github.com/apache/pinot/blob/c6e8cab1fa2e35f0d4e39a5a5b5d95886e76aa45/pinot-s

(pinot) 01/01: Initial POC code for hybrid table

2024-03-05 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch full-auto-abstraction in repository https://gitbox.apache.org/repos/asf/pinot.git commit 025f388f44a66d44bdcbd7d9ccc707bc1ebd5fde Author: jlli_LinkedIn AuthorDate: Mon Feb 12 23:36:47 2024 -0800

(pinot) branch full-auto-abstraction created (now 025f388f44)

2024-03-05 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch full-auto-abstraction in repository https://gitbox.apache.org/repos/asf/pinot.git at 025f388f44 Initial POC code for hybrid table This branch includes the following new commits: new 02

Re: [I] Segment Compaction tasks fail when snapshot file doesn't exist on lagging server [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang closed issue #12535: Segment Compaction tasks fail when snapshot file doesn't exist on lagging server URL: https://github.com/apache/pinot/issues/12535 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[PR] Bump com.squareup.okio:okio from 2.10.0 to 3.4.0 [pinot]

2024-03-05 Thread via GitHub
dependabot[bot] opened a new pull request, #12563: URL: https://github.com/apache/pinot/pull/12563 Bumps [com.squareup.okio:okio](https://github.com/square/okio) from 2.10.0 to 3.4.0. Changelog Sourced from https://github.com/square/okio/blob/master/CHANGELOG.md";>com.squareup.okio

(pinot) branch dependabot/maven/com.squareup.okio-okio-3.4.0 created (now c6d7ba892c)

2024-03-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/com.squareup.okio-okio-3.4.0 in repository https://gitbox.apache.org/repos/asf/pinot.git at c6d7ba892c Bump com.squareup.okio:okio from 2.10.0 to 3.4.0 No new revisio

(pinot) branch master updated: Upgrade hadoop version to 3.3.6 (#12561)

2024-03-05 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 c6e8cab1fa Upgrade hadoop version to 3.3.6 (#125

Re: [PR] Upgrade hadoop version to 3.3.6 [pinot]

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

Re: [PR] bugfix: do not move src ByteBuffer position for LZ4 length prefixed decompress [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on PR #12539: URL: https://github.com/apache/pinot/pull/12539#issuecomment-1979794221 Trying to understand the problem better here. Why would it cause problem when the source position is moved? It is allowed for `decompress()` to move the position of the compressed bu

(pinot) branch master updated (964ff0ce85 -> 0420a9b66c)

2024-03-05 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 964ff0ce85 Fix environment variables not applied when creating table (#12560) add 0420a9b66c Enter segment preloa

Re: [PR] Enter segment preloading at partition level [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang merged PR #12451: URL: https://github.com/apache/pinot/pull/12451 -- 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] [feature] use async methods to read from Pulsar topic [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on PR #12061: URL: https://github.com/apache/pinot/pull/12061#issuecomment-1979756517 There are test failures. @JeffBolle Can you take a look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [I] 1.0.1 release timeline [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang closed issue #12428: 1.0.1 release timeline URL: https://github.com/apache/pinot/issues/12428 -- 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-mai

[PR] aws sdk 2.25.3 [pinot]

2024-03-05 Thread via GitHub
sullis opened a new pull request, #12562: URL: https://github.com/apache/pinot/pull/12562 https://github.com/aws/aws-sdk-java-v2/releases/tag/2.25.3 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [PR] Modify consumingSegmentsInfo endpoint to indicate how many servers failed [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on code in PR #12523: URL: https://github.com/apache/pinot/pull/12523#discussion_r1513554428 ## pinot-controller/src/main/java/org/apache/pinot/controller/util/ConsumingSegmentInfoReader.java: ## @@ -132,7 +133,8 @@ private Map> getConsumingSegmentsInfoFr

(pinot) 01/01: Initial POC code for hybrid table

2024-03-05 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a commit to branch full-auto-poc in repository https://gitbox.apache.org/repos/asf/pinot.git commit 6781a10bf41f168aa80a48a823645446f228fcd7 Author: jlli_LinkedIn AuthorDate: Mon Feb 12 23:36:47 2024 -0800 Initi

(pinot) branch full-auto-poc updated (37dc958fbc -> 6781a10bf4)

2024-03-05 Thread jlli
This is an automated email from the ASF dual-hosted git repository. jlli pushed a change to branch full-auto-poc in repository https://gitbox.apache.org/repos/asf/pinot.git discard 37dc958fbc Initial POC code for hybrid table new 6781a10bf4 Initial POC code for hybrid table This update ad

Re: [PR] Upgrade hadoop version to 3.3.6 [pinot]

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

[PR] Upgrade hadoop version to 3.3.6 [pinot]

2024-03-05 Thread via GitHub
xiangfu0 opened a new pull request, #12561: URL: https://github.com/apache/pinot/pull/12561 Upgrade hadoop version to 3.3.6 -- 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.

Re: [PR] validate DropwizardMetricName equals [pinot]

2024-03-05 Thread via GitHub
codecov-commenter commented on PR #12559: URL: https://github.com/apache/pinot/pull/12559#issuecomment-1979532391 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/12559?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: Fix environment variables not applied when creating table (#12560)

2024-03-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 964ff0ce85 Fix environment variables not applied

Re: [I] Pinot Kafka config environment variables no longer being applied [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang closed issue #12556: Pinot Kafka config environment variables no longer being applied URL: https://github.com/apache/pinot/issues/12556 -- 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

Re: [PR] Fix environment variables not applied when creating table [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang merged PR #12560: URL: https://github.com/apache/pinot/pull/12560 -- 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] Fix issue with startree index metadata loading for columns with '__' in name [pinot]

2024-03-05 Thread via GitHub
codecov-commenter commented on PR #12554: URL: https://github.com/apache/pinot/pull/12554#issuecomment-1979466904 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/12554?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: Support empty strings as json nodes^ (#12555)

2024-03-05 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 12060f7770 Support empty strings as json nodes^ (

Re: [PR] Support empty strings as json nodes^ [pinot]

2024-03-05 Thread via GitHub
gortiz merged PR #12555: URL: https://github.com/apache/pinot/pull/12555 -- 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] exclude .mvn directory from source assembly [pinot]

2024-03-05 Thread via GitHub
xiangfu0 merged PR #12558: URL: https://github.com/apache/pinot/pull/12558 -- 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: exclude .mvn directory from source assembly (#12558)

2024-03-05 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 fc776e77c5 exclude .mvn directory from source as

Re: [PR] add test cases in RequestUtilsTest [pinot]

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

Re: [I] Pinot Kafka config environment variables no longer being applied [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang commented on issue #12556: URL: https://github.com/apache/pinot/issues/12556#issuecomment-1979426519 Thanks for raising the issue. Verified that this is a bug, and is fixed with #12560 -- This is an automated message from the Apache Git Service. To respond to the message, pl

[PR] Fix environment variables not applied when creating table [pinot]

2024-03-05 Thread via GitHub
Jackie-Jiang opened a new pull request, #12560: URL: https://github.com/apache/pinot/pull/12560 Fix #12556 -- 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

[PR] validate DropwizardMetricName equals [pinot]

2024-03-05 Thread via GitHub
sullis opened a new pull request, #12559: URL: https://github.com/apache/pinot/pull/12559 We use the equalsverifier library to validate 'equals' method https://jqno.nl/equalsverifier/ -- This is an automated message from the Apache Git Service. To respond to the message,

[PR] exclude .mvn directory from source assembly [pinot]

2024-03-05 Thread via GitHub
xiangfu0 opened a new pull request, #12558: URL: https://github.com/apache/pinot/pull/12558 Instructions: 1. The PR has to be tagged with at least one of the following labels (*): 1. `feature` 2. `bugfix` 3. `performance` 4. `ui` 5. `backward-incompat` 6

Re: [PR] Updated MinionTaskUtils.getValidDocIdsBitmap to send requests to other servers when an exception occurs [pinot]

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

(pinot) branch master updated: replaced getServer with getServers (#12545)

2024-03-05 Thread snlee
This is an automated email from the ASF dual-hosted git repository. snlee 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 96fdacbf72 replaced getServer with getServers (#12

[PR] add test cases in RequestUtilsTest [pinot]

2024-03-05 Thread via GitHub
sullis opened a new pull request, #12557: URL: https://github.com/apache/pinot/pull/12557 (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-mai

[PR] Support empty strings as json nodes^ [pinot]

2024-03-05 Thread via GitHub
gortiz opened a new pull request, #12555: URL: https://github.com/apache/pinot/pull/12555 A PR to change the flatten function so the Java empty String (`""`) is treated as null -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Fix issue with startree index metadata loading for columns with '__' in name [pinot]

2024-03-05 Thread via GitHub
gortiz commented on PR #12554: URL: https://github.com/apache/pinot/pull/12554#issuecomment-1978632795 Could you add some test? I think the code is fine, but just to have some protection versus regressions in the future -- This is an automated message from the Apache Git Service. To respo

[PR] Fix issue with startree index metadata loading for columns with '__' in name [pinot]

2024-03-05 Thread via GitHub
saurabhd336 opened a new pull request, #12554: URL: https://github.com/apache/pinot/pull/12554 https://github.com/apache/pinot/pull/12164/files introduced shared aggregation which assumes all metadata entries with '__' in the name are AggregationFunctionColumnPair. This breaks startr

(pinot) branch dependabot/maven/nl.jqno.equalsverifier-equalsverifier-3.15.8 created (now 8ca6e96b97)

2024-03-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/nl.jqno.equalsverifier-equalsverifier-3.15.8 in repository https://gitbox.apache.org/repos/asf/pinot.git at 8ca6e96b97 Bump nl.jqno.equalsverifier:equalsverifier from

[PR] Bump nl.jqno.equalsverifier:equalsverifier from 3.14.1 to 3.15.8 [pinot]

2024-03-05 Thread via GitHub
dependabot[bot] opened a new pull request, #12553: URL: https://github.com/apache/pinot/pull/12553 Bumps [nl.jqno.equalsverifier:equalsverifier](https://github.com/jqno/equalsverifier) from 3.14.1 to 3.15.8. Release notes Sourced from https://github.com/jqno/equalsverifier/release

Re: [I] Consumption stops after a specified number of rows [pinot]

2024-03-05 Thread via GitHub
vineethvp commented on issue #12474: URL: https://github.com/apache/pinot/issues/12474#issuecomment-1978234678 Related to https://github.com/apache/pinot/issues/11407#issuecomment-1978230317 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [I] Consumption stops after a specified number of rows [pinot]

2024-03-05 Thread via GitHub
vineethvp closed issue #12474: Consumption stops after a specified number of rows URL: https://github.com/apache/pinot/issues/12474 -- 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: [I] Consumption stops after a specified number of rows [pinot]

2024-03-05 Thread via GitHub
vineethvp commented on issue #12474: URL: https://github.com/apache/pinot/issues/12474#issuecomment-1978233079 It is related to -- 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: [I] Servers can't consume messages due to permission denied error in the endpoint '/segmentConsumed' when ACL is enabled [pinot]

2024-03-05 Thread via GitHub
vineethvp commented on issue #11407: URL: https://github.com/apache/pinot/issues/11407#issuecomment-1978230317 @KKcorps @zhtaoxiang Any updated on this? -- 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