svn commit: r67672 - /dev/pinot/apache-pinot-1.1.0-rc0/

2024-03-02 Thread vvivekiyer
Author: vvivekiyer Date: Sun Mar 3 06:18:35 2024 New Revision: 67672 Log: Update apache-pinot-1.1.0-rc0 Added: dev/pinot/apache-pinot-1.1.0-rc0/ dev/pinot/apache-pinot-1.1.0-rc0/apache-pinot-1.1.0-bin.tar.gz (with props) dev/pinot/apache-pinot-1.1.0-rc0/apache-pinot-1.1.0-bin.tar.

(pinot) branch release-1.1.0-rc updated: [maven-release-plugin] prepare for next development iteration

2024-03-02 Thread vvivekiyer
This is an automated email from the ASF dual-hosted git repository. vvivekiyer pushed a commit to branch release-1.1.0-rc in repository https://gitbox.apache.org/repos/asf/pinot.git The following commit(s) were added to refs/heads/release-1.1.0-rc by this push: new d3d3089e37 [maven-release

(pinot) annotated tag release-1.1.0-rc0 updated (b10b4dd249 -> 5c1f08d2cc)

2024-03-02 Thread vvivekiyer
This is an automated email from the ASF dual-hosted git repository. vvivekiyer pushed a change to annotated tag release-1.1.0-rc0 in repository https://gitbox.apache.org/repos/asf/pinot.git *** WARNING: tag release-1.1.0-rc0 was modified! *** from b10b4dd249 (commit) to 5c1f08d2cc (ta

(pinot) branch release-1.1.0-rc updated: [maven-release-plugin] prepare release release-1.1.0-rc0

2024-03-02 Thread vvivekiyer
This is an automated email from the ASF dual-hosted git repository. vvivekiyer pushed a commit to branch release-1.1.0-rc in repository https://gitbox.apache.org/repos/asf/pinot.git The following commit(s) were added to refs/heads/release-1.1.0-rc by this push: new b10b4dd249 [maven-release

(pinot) branch release-1.1.0-rc updated (64f9d792f1 -> ce94496e2e)

2024-03-02 Thread vvivekiyer
This is an automated email from the ASF dual-hosted git repository. vvivekiyer pushed a change to branch release-1.1.0-rc in repository https://gitbox.apache.org/repos/asf/pinot.git discard 64f9d792f1 [maven-release-plugin] prepare for next development iteration discard 49ee95a75b [maven-rele

(pinot) branch local-release-1.1.0-rc deleted (was ce94496e2e)

2024-03-02 Thread vvivekiyer
This is an automated email from the ASF dual-hosted git repository. vvivekiyer pushed a change to branch local-release-1.1.0-rc in repository https://gitbox.apache.org/repos/asf/pinot.git was ce94496e2e Update license & notice for release 1.1.0 The revisions that were on this branch are st

(pinot) branch local-release-1.1.0-rc created (now ce94496e2e)

2024-03-02 Thread vvivekiyer
This is an automated email from the ASF dual-hosted git repository. vvivekiyer pushed a change to branch local-release-1.1.0-rc in repository https://gitbox.apache.org/repos/asf/pinot.git at ce94496e2e Update license & notice for release 1.1.0 No new revisions were added by this update.

(pinot) annotated tag release-1.1.0-rc0 deleted (was 37bd7be690)

2024-03-02 Thread vvivekiyer
This is an automated email from the ASF dual-hosted git repository. vvivekiyer pushed a change to annotated tag release-1.1.0-rc0 in repository https://gitbox.apache.org/repos/asf/pinot.git *** WARNING: tag release-1.1.0-rc0 was deleted! *** tag was 37bd7be690 The revisions that were on th

svn commit: r67670 - /dev/pinot/apache-pinot-1.1.0-rc0/

2024-03-02 Thread vvivekiyer
Author: vvivekiyer Date: Sun Mar 3 01:00:00 2024 New Revision: 67670 Log: Delete release-1.1.0-rc0 Removed: dev/pinot/apache-pinot-1.1.0-rc0/ - To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional co

Re: [PR] Potential bug fix for partial-upsert with sorted-indexes [pinot]

2024-03-02 Thread via GitHub
ankitsultana commented on code in PR #12544: URL: https://github.com/apache/pinot/pull/12544#discussion_r1510117525 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java: ## @@ -640,7 +640,10 @@ protected static Iterat

Re: [PR] Potential bug fix for partial-upsert with sorted-indexes [pinot]

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

[PR] Potential bug fix for partial-upsert with sorted-indexes [pinot]

2024-03-02 Thread via GitHub
tibrewalpratik17 opened a new pull request, #12544: URL: https://github.com/apache/pinot/pull/12544 label: `bugfix` Potential fix for #12397 As called out in the issue: ``` With Partial Upserts, say we had 4 events for a given primary-key in the Mutable Segment: R0,

Re: [PR] remove duplicate scala.version [pinot]

2024-03-02 Thread via GitHub
sullis commented on PR #12543: URL: https://github.com/apache/pinot/pull/12543#issuecomment-1974926663 ready to merge -- 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 un

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

2024-03-02 Thread via GitHub
richardstartin commented on PR #12539: URL: https://github.com/apache/pinot/pull/12539#issuecomment-1974909646 This is a good catch. I have a couple of remarks: 1. It would be good to add a test because this is a subtle bug that has been hiding in plain sight for over two years, and it c

Re: [PR] Added /uptime and /start-time for all components. Addressed comments and fixed tests from pull request 12389. [pinot]

2024-03-02 Thread via GitHub
suyashpatel98 commented on PR #12512: URL: https://github.com/apache/pinot/pull/12512#issuecomment-1974897213 ForwardIndexHandlerTest.testAddOtherIndexForForwardIndexDisabledColumn() is a flaky test. There is an existing open issue - #11928 -- This is an automated message from the Apache

Re: [PR] remove duplicate scala.version [pinot]

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

Re: [PR] mockito 5.11.0 [pinot]

2024-03-02 Thread via GitHub
sullis commented on PR #12536: URL: https://github.com/apache/pinot/pull/12536#issuecomment-1974877481 ready to merge -- 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 uns

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

2024-03-02 Thread via GitHub
aditya0811 commented on issue #12367: URL: https://github.com/apache/pinot/issues/12367#issuecomment-1974871309 Hey Jackie, sorry I am not able to follow. Lets say `_floatSVValues[INDEX_TO_COMPARE] = 0.1f`. In the case block in L721 in `CaseTransformFunctionTest` we are considering this

Re: [PR] maven-compiler-plugin 3.12.1 [pinot]

2024-03-02 Thread via GitHub
sullis commented on PR #12540: URL: https://github.com/apache/pinot/pull/12540#issuecomment-1974870365 Ready to merge -- 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 uns

[PR] remove duplicate scala.version [pinot]

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

Re: [PR] s3mock 3.5.1 [pinot]

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

Re: [PR] scala-maven-plugin 4.8.1 [pinot]

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

Re: [PR] maven-compiler-plugin 3.12.1 [pinot]

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

[PR] s3mock 3.5.1 [pinot]

2024-03-02 Thread via GitHub
sullis opened a new pull request, #12542: URL: https://github.com/apache/pinot/pull/12542 https://github.com/adobe/S3Mock https://mvnrepository.com/artifact/com.adobe.testing/s3mock-testcontainers -- This is an automated message from the Apache Git Service. To respond to the

[PR] scala-maven-plugin 4.8.1 [pinot]

2024-03-02 Thread via GitHub
sullis opened a new pull request, #12541: URL: https://github.com/apache/pinot/pull/12541 https://mvnrepository.com/artifact/net.alchim31.maven/scala-maven-plugin -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[PR] maven-compiler-plugin 3.12.1 [pinot]

2024-03-02 Thread via GitHub
sullis opened a new pull request, #12540: URL: https://github.com/apache/pinot/pull/12540 https://maven.apache.org/plugins/maven-compiler-plugin/ -- 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

Re: [PR] DRAFT: Adding SHOW TABLES/DESCRIBE table support [pinot]

2024-03-02 Thread via GitHub
xiangfu0 commented on code in PR #12537: URL: https://github.com/apache/pinot/pull/12537#discussion_r1509943974 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotQueryResource.java: ## @@ -243,10 +246,17 @@ private String getQueryResponse(String qu

Re: [PR] DRAFT: Adding SHOW TABLES/DESCRIBE table support [pinot]

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

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

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

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

2024-03-02 Thread via GitHub
itschrispeck opened a new pull request, #12539: URL: https://github.com/apache/pinot/pull/12539 Addresses https://github.com/apache/pinot/issues/12534 Based on my understanding, calling `_decompressor.decompress(compressedInput, decompressedOutput)` is problematic because it [moves

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

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