vrajat closed pull request #15297: Model Hybrid Table behaviour in SSE query
planning using HybridTable class
URL: https://github.com/apache/pinot/pull/15297
--
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 t
vrajat commented on PR #15297:
URL: https://github.com/apache/pinot/pull/15297#issuecomment-2791702582
Closing. Moved to #15388
--
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
abhishekbafna commented on code in PR #15388:
URL: https://github.com/apache/pinot/pull/15388#discussion_r2036469268
##
pinot-core/src/main/java/org/apache/pinot/core/transport/ImplicitHybridTableRouteInfo.java:
##
@@ -0,0 +1,135 @@
+/**
+ * Licensed to the Apache Software Found
KKcorps commented on code in PR #15495:
URL: https://github.com/apache/pinot/pull/15495#discussion_r2036418450
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java:
##
@@ -2573,66 +2573,37 @@ public Set getSegments
abhishekbafna commented on code in PR #15388:
URL: https://github.com/apache/pinot/pull/15388#discussion_r2036469268
##
pinot-core/src/main/java/org/apache/pinot/core/transport/ImplicitHybridTableRouteInfo.java:
##
@@ -0,0 +1,135 @@
+/**
+ * Licensed to the Apache Software Found
wirybeaver commented on PR #15203:
URL: https://github.com/apache/pinot/pull/15203#issuecomment-2791497037
https://github.com/apache/pinot/commit/72d4aa046d01d2c2a50f8f2479f7de9fc7c0113a
Fail on Pinot Tests / Pinot Integration Test Set 1 (temurin-11) with skip
bytebuffers:false (pull_req
vvivekiyer commented on issue #14551:
URL: https://github.com/apache/pinot/issues/14551#issuecomment-2791489969
@ankitsultana Yes, this would be a bigger scope. We intend to create a
scheduler as well for this but it will come after the MVP implementation.
We are currently benchmarki
This is an automated email from the ASF dual-hosted git repository.
kharekartik 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 9145e92f65 Fix committing segments list (#15
KKcorps merged PR #15495:
URL: https://github.com/apache/pinot/pull/15495
--
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.apac
KKcorps commented on code in PR #15495:
URL: https://github.com/apache/pinot/pull/15495#discussion_r2036418450
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java:
##
@@ -2573,66 +2573,37 @@ public Set getSegments
codecov-commenter commented on PR #15511:
URL: https://github.com/apache/pinot/pull/15511#issuecomment-2791448486
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15511?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
wirybeaver commented on PR #15203:
URL: https://github.com/apache/pinot/pull/15203#issuecomment-2791440965
https://github.com/apache/pinot/commit/72d4aa046d01d2c2a50f8f2479f7de9fc7c0113a
The previous ErrorCodeIntegration test which always failed became
successful. Only one test case
himanish-star commented on code in PR #15511:
URL: https://github.com/apache/pinot/pull/15511#discussion_r2036402556
##
pinot-controller/src/main/resources/app/components/Homepage/Operations/RebalanceServer/RebalanceServerConfigurationOptions/RebalanceServerConfigurationOptionInteger.ts
himanish-star opened a new pull request, #15511:
URL: https://github.com/apache/pinot/pull/15511
### Description
This PR adds the following fixes:
1. Adds a `back` button to go back after `Dry Run` or `Rebalance` action.
2. Adds boxes around `segment map` and `instance map`
3.
wirybeaver commented on PR #15203:
URL: https://github.com/apache/pinot/pull/15203#issuecomment-2791404145
RCA:
In my diff, I introduced a static singleton instance of the
enableServerInstance map. This approach doesn’t cause issues in regular
production environments since there’s ty
J-HowHuang commented on code in PR #15496:
URL: https://github.com/apache/pinot/pull/15496#discussion_r2036153944
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java:
##
@@ -77,7 +78,9 @@ public Map
check(PreCheckCo
Jackie-Jiang closed issue #15179: Extend POSSIBLE_QUERY_OPTIONS in
PinotConnection to all query level options
URL: https://github.com/apache/pinot/issues/15179
--
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
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/org.mockito-mockito-core-5.17.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 611f2b44e2 Bump org.mockito:mockito-core from 5.16.1 to 5.17.0
The revision
Jackie-Jiang commented on code in PR #15436:
URL: https://github.com/apache/pinot/pull/15436#discussion_r2036270631
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalanceLogger.java:
##
@@ -0,0 +1,61 @@
+/**
+ * Licensed to the Apache S
J-HowHuang commented on code in PR #15436:
URL: https://github.com/apache/pinot/pull/15436#discussion_r2036212811
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalanceLogger.java:
##
@@ -0,0 +1,58 @@
+/**
+ * Licensed to the Apache Sof
Jackie-Jiang commented on code in PR #15496:
URL: https://github.com/apache/pinot/pull/15496#discussion_r2036229108
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java:
##
@@ -77,7 +78,9 @@ public Map
check(PreCheck
chrajeshbabu commented on issue #15472:
URL: https://github.com/apache/pinot/issues/15472#issuecomment-2784083271
Yes @xiangfu0 . Am working on the change will submit PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
J-HowHuang commented on code in PR #15436:
URL: https://github.com/apache/pinot/pull/15436#discussion_r2036212811
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalanceLogger.java:
##
@@ -0,0 +1,58 @@
+/**
+ * Licensed to the Apache Sof
Jackie-Jiang commented on code in PR #15496:
URL: https://github.com/apache/pinot/pull/15496#discussion_r2036158328
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java:
##
@@ -77,7 +78,9 @@ public Map
check(PreCheck
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/com.gradle-develocity-maven-extension-2.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was fcb7182522 Bump com.gradle:develocity-maven-extension from 1.23.2
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/pulsar.version-4.0.4
in repository https://gitbox.apache.org/repos/asf/pinot.git
was fa86a1c2cb Bump pulsar.version from 4.0.3 to 4.0.4
The revisions that were on this bran
J-HowHuang commented on code in PR #15496:
URL: https://github.com/apache/pinot/pull/15496#discussion_r2036161386
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java:
##
@@ -77,7 +78,9 @@ public Map
check(PreCheckCo
Jackie-Jiang commented on code in PR #15436:
URL: https://github.com/apache/pinot/pull/15436#discussion_r2036153737
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalanceLogger.java:
##
@@ -0,0 +1,58 @@
+/**
+ * Licensed to the Apache S
Jackie-Jiang commented on code in PR #15496:
URL: https://github.com/apache/pinot/pull/15496#discussion_r2036146895
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/DefaultRebalancePreChecker.java:
##
@@ -77,7 +78,9 @@ public Map
check(PreCheck
Jackie-Jiang merged PR #15431:
URL: https://github.com/apache/pinot/pull/15431
--
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
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 bb29f0ab7a Bump com.gradle:develocity-maven-extension from 1.23.2 to
2.0 (#15507)
add 2bbae4d03f Disables catch u
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 01c62ef0f8 Bump pulsar.version from 4.0.3 to 4.0.4 (#15506)
add bb29f0ab7a Bump com.gradle:develocity-maven-extens
Jackie-Jiang merged PR #15507:
URL: https://github.com/apache/pinot/pull/15507
--
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
Jackie-Jiang merged PR #15506:
URL: https://github.com/apache/pinot/pull/15506
--
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
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 83693ff2f7 Refactor SegmentZKMetadata update logic during segment
commit (#15482)
add 01c62ef0f8 Bump pulsar.vers
Jackie-Jiang merged PR #15482:
URL: https://github.com/apache/pinot/pull/15482
--
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
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.31.16
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 607e5e5fd1 Bump software.amazon.awssdk:bom from 2.31.15 to 2.31.16
The r
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 d7b17630a6 [Issue 12676 - Part 2.2] Define versions from submodule
POMs in root POM (#15501)
add 83693ff2f7 Refac
codecov-commenter commented on PR #15510:
URL: https://github.com/apache/pinot/pull/15510#issuecomment-2790830185
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15510?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
jitendrakr88 commented on PR #15498:
URL: https://github.com/apache/pinot/pull/15498#issuecomment-2790748196
> Does it work if we simply trim off the white space and optional `;` at the
end of the query and then perform the regex match? We don't want to bring back
the vulnerability of code
somandal opened a new pull request, #15510:
URL: https://github.com/apache/pinot/pull/15510
As part of PR https://github.com/apache/pinot/pull/15266, progress stats
were enhanced for Table rebalance. A scenario (details:
https://github.com/apache/pinot/pull/15266#issuecomment-2779453530) wa
codecov-commenter commented on PR #15509:
URL: https://github.com/apache/pinot/pull/15509#issuecomment-2790560302
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15509?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
somandal commented on code in PR #15436:
URL: https://github.com/apache/pinot/pull/15436#discussion_r2035821939
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java:
##
@@ -1333,16 +1352,18 @@ private IdealState waitForExternalV
siddharthteotia merged PR #15501:
URL: https://github.com/apache/pinot/pull/15501
--
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...@pi
somandal commented on code in PR #15436:
URL: https://github.com/apache/pinot/pull/15436#discussion_r2035821939
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancer.java:
##
@@ -1333,16 +1352,18 @@ private IdealState waitForExternalV
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-4.0.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
discard e5d51c9065 Bump com.adobe.testing:s3mock-testcontainers fr
J-HowHuang opened a new pull request, #15509:
URL: https://github.com/apache/pinot/pull/15509
## Description
In most cases, particuarly for tables using tiered storage, this
segmentRelocator is used to move committed segments to remote tier. Given this
`includeConsuming=false` and `m
This is an automated email from the ASF dual-hosted git repository.
siddteotia 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 d7b17630a6 [Issue 12676 - Part 2.2] Define ve
siddharthteotia commented on code in PR #15501:
URL: https://github.com/apache/pinot/pull/15501#discussion_r2035817787
##
pinot-plugins/pinot-stream-ingestion/pinot-kinesis/pom.xml:
##
@@ -32,7 +32,6 @@
https://pinot.apache.org/
${basedir}/../../..
-0.2.23
Revie
siddharthteotia commented on code in PR #15501:
URL: https://github.com/apache/pinot/pull/15501#discussion_r2035817056
##
pinot-plugins/pinot-file-system/pinot-s3/pom.xml:
##
@@ -32,7 +32,6 @@
https://pinot.apache.org
${basedir}/../../..
-2.17.0
Review Comment:
siddharthteotia commented on code in PR #15501:
URL: https://github.com/apache/pinot/pull/15501#discussion_r2035815132
##
pinot-plugins/pinot-stream-ingestion/pinot-kinesis/pom.xml:
##
@@ -32,7 +32,6 @@
https://pinot.apache.org/
${basedir}/../../..
-0.2.23
Revie
itschrispeck commented on issue #15503:
URL: https://github.com/apache/pinot/issues/15503#issuecomment-2790379774
It seems like a bug if a column would dictionary encoded if `"encodingType":
"RAW"` is set. We attempt to [disable dictionary based on encodingType
here](https://github.com/apac
deepthi912 commented on code in PR #15335:
URL: https://github.com/apache/pinot/pull/15335#discussion_r2035254482
##
pinot-core/src/main/java/org/apache/pinot/core/transport/DirectOOMServerHandler.java:
##
@@ -0,0 +1,113 @@
+/**
+ * Licensed to the Apache Software Foundation (AS
codecov-commenter commented on PR #15508:
URL: https://github.com/apache/pinot/pull/15508#issuecomment-2790096072
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15508?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
bziobrowski commented on code in PR #15508:
URL: https://github.com/apache/pinot/pull/15508#discussion_r2035559785
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/segment/index/JsonIndexTest.java:
##
@@ -44,7 +44,6 @@
import org.apache.pinot.spi.utils.JsonUt
bziobrowski commented on code in PR #15508:
URL: https://github.com/apache/pinot/pull/15508#discussion_r2035549838
##
pinot-core/src/test/java/org/apache/pinot/core/operator/transform/function/BaseTransformFunctionTest.java:
##
@@ -311,18 +311,7 @@ public void setUp()
.
bziobrowski opened a new pull request, #15508:
URL: https://github.com/apache/pinot/pull/15508
Work in progress.
PR adds OSS changes necessary to support startree json index.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/org.codehaus.plexus-plexus-classworlds-2.9.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was dd66a7f33e Bump org.codehaus.plexus:plexus-classworlds from 2.8
Jackie-Jiang merged PR #15477:
URL: https://github.com/apache/pinot/pull/15477
--
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
dependabot[bot] closed pull request #15490: Bump software.amazon.awssdk:bom
from 2.31.16 to 2.31.17
URL: https://github.com/apache/pinot/pull/15490
--
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 th
codecov-commenter commented on PR #15507:
URL: https://github.com/apache/pinot/pull/15507#issuecomment-2789566403
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15507?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
codecov-commenter commented on PR #15506:
URL: https://github.com/apache/pinot/pull/15506#issuecomment-2789562236
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15506?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
codecov-commenter commented on PR #15505:
URL: https://github.com/apache/pinot/pull/15505#issuecomment-2789561419
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15505?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
codecov-commenter commented on PR #15502:
URL: https://github.com/apache/pinot/pull/15502#issuecomment-2788725841
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15502?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
dependabot[bot] opened a new pull request, #15507:
URL: https://github.com/apache/pinot/pull/15507
Bumps com.gradle:develocity-maven-extension from 1.23.2 to 2.0.
[ were added to refs/heads/master by this push:
new f7fe95db97 Extend POSSIBLE_QUERY_OPTIONS in Pinot
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 fc8c770fcd STP-3739: Segment files batch deletio
xiangfu0 merged PR #15473:
URL: https://github.com/apache/pinot/pull/15473
--
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
albertobastos opened a new pull request, #15502:
URL: https://github.com/apache/pinot/pull/15502
SSE includes an optimization for queries where filter can be determined to
be always false. When that happens, the response is empty and no actual calls
are sent to the Pinot Servers.
How
77 matches
Mail list logo