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 d1a71d8518 Fix PerfBenchmarkRunner to skip the tm
Jackie-Jiang merged PR #9298:
URL: https://github.com/apache/pinot/pull/9298
--
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.a
xiangfu0 commented on PR #9294:
URL: https://github.com/apache/pinot/pull/9294#issuecomment-1231215626
Can you add a sample data file with decimal field and test the file is
correctly parsed.
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
snleee closed issue #9297: Flaky test for
`RealtimeNonReplicaGroupSegmentAssignmentTest.testRelocateCompletedSegments()`
URL: https://github.com/apache/pinot/issues/9297
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
snleee commented on issue #9297:
URL: https://github.com/apache/pinot/issues/9297#issuecomment-1231181492
#9253
@Jackie-Jiang already created the same issue and he actually added the above
test to prove the flaky test. closing this ticket and we can address the fix in
the above.
--
Th
snleee commented on issue #9297:
URL: https://github.com/apache/pinot/issues/9297#issuecomment-1231178474
For this test, we run Helix's
`AutoRebalanceStrategy.computePartitionAssignment()` for completed segments. We
pass `node.capacity = targetSize (which we compute from "distFloor"`; howev
siddharthteotia merged PR #9282:
URL: https://github.com/apache/pinot/pull/9282
--
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...@pino
gortiz commented on issue #9292:
URL: https://github.com/apache/pinot/issues/9292#issuecomment-1231168693
Also, right now brokers temporally store the data from servers in a hash
map, which implies to hash is twice (once when it is being read from the server
and another one when data from d
snleee commented on issue #9297:
URL: https://github.com/apache/pinot/issues/9297#issuecomment-1231167798
Most of the time, the test passes with the `[29, 30, 30, 30, 29, 29, 30, 29,
29, 29]`
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
snleee opened a new issue, #9297:
URL: https://github.com/apache/pinot/issues/9297
```
java.lang.AssertionError: Expect at least: 29 segments assigned per
instance, got: [30, 30, 30, 30, 29, 29, 29, 29, 30, 28]
Expected :true
Actual :false
at org.testng.Assert
snleee commented on code in PR #9295:
URL: https://github.com/apache/pinot/pull/9295#discussion_r957981334
##
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/SegmentUploadIntegrationTest.java:
##
@@ -81,10 +89,15 @@ protected List getBloomFilterColumns()
yuanbenson closed pull request #9295: Enable Consistent Data Push for
Standalone Segment Push Job Runners
URL: https://github.com/apache/pinot/pull/9295
--
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
This is an automated email from the ASF dual-hosted git repository.
sajjad 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 a273af0262 Add capabilities to ingest from anothe
sajjad-moradi merged PR #9289:
URL: https://github.com/apache/pinot/pull/9289
--
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.
siddharthteotia commented on code in PR #9296:
URL: https://github.com/apache/pinot/pull/9296#discussion_r957847415
##
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/CastTransformFunction.java:
##
@@ -84,6 +84,22 @@ public void init(List arguments,
M
siddharthteotia commented on code in PR #9296:
URL: https://github.com/apache/pinot/pull/9296#discussion_r957846649
##
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/CastTransformFunction.java:
##
@@ -84,6 +84,22 @@ public void init(List arguments,
M
SabrinaZhaozyf opened a new pull request, #9296:
URL: https://github.com/apache/pinot/pull/9296
Label: `feature`
This PR adds support for Cast transform function for mv columns.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
Jackie-Jiang commented on code in PR #9282:
URL: https://github.com/apache/pinot/pull/9282#discussion_r957828740
##
pinot-core/src/main/java/org/apache/pinot/core/plan/maker/InstancePlanMakerImplV2.java:
##
@@ -273,14 +273,13 @@ public Plan makeStreamingInstancePlan(List
indexS
npawar commented on code in PR #9289:
URL: https://github.com/apache/pinot/pull/9289#discussion_r957827576
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotRealtimeTableResource.java:
##
@@ -77,14 +77,29 @@ public Response pauseConsumption(
@PO
Jackie-Jiang commented on code in PR #9282:
URL: https://github.com/apache/pinot/pull/9282#discussion_r957827047
##
pinot-core/src/main/java/org/apache/pinot/core/operator/InstanceResponseOperator.java:
##
@@ -34,24 +34,21 @@
public class InstanceResponseOperator extends
B
Jackie-Jiang commented on code in PR #9282:
URL: https://github.com/apache/pinot/pull/9282#discussion_r957812873
##
pinot-core/src/main/java/org/apache/pinot/core/operator/AcquireReleaseColumnsSegmentOperator.java:
##
@@ -58,8 +58,8 @@ public AcquireReleaseColumnsSegmentOperator
yuanbenson opened a new pull request, #9295:
URL: https://github.com/apache/pinot/pull/9295
### Description:
This PR addresses https://github.com/apache/pinot/issues/9268 for the
standalone execution framework segment push jobs:
`SegmentMetadataPushJobRunner`, `SegmentTarPushJobRunner`,
KKcorps commented on PR #9288:
URL: https://github.com/apache/pinot/pull/9288#issuecomment-1230831952
It is failing in some cases like local environment but multi threaded.
Working on fixing those post which we can merge.
--
This is an automated message from the Apache Git Service.
To res
walterddr commented on code in PR #9282:
URL: https://github.com/apache/pinot/pull/9282#discussion_r957726100
##
pinot-core/src/main/java/org/apache/pinot/core/plan/maker/InstancePlanMakerImplV2.java:
##
@@ -273,14 +273,13 @@ public Plan makeStreamingInstancePlan(List
indexSegm
This is an automated email from the ASF dual-hosted git repository.
sajjad 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 5ecca80cc8 Force commit consuming segments (#9197
sajjad-moradi merged PR #9197:
URL: https://github.com/apache/pinot/pull/9197
--
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.
sajjad-moradi commented on code in PR #9289:
URL: https://github.com/apache/pinot/pull/9289#discussion_r957671202
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java:
##
@@ -1074,6 +1077,15 @@ IdealState ensureAll
sajjad-moradi commented on code in PR #9289:
URL: https://github.com/apache/pinot/pull/9289#discussion_r957671413
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java:
##
@@ -1144,21 +1156,33 @@ IdealState ensureAl
This is an automated email from the ASF dual-hosted git repository.
sajjad 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 8767437931 Fix issue with auth AccessType in Sche
sajjad-moradi merged PR #9293:
URL: https://github.com/apache/pinot/pull/9293
--
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.
npawar commented on PR #9114:
URL: https://github.com/apache/pinot/pull/9114#issuecomment-1230676758
> @npawar Changes to user doc has been merged here
([pinot-contrib/pinot-docs#109](https://github.com/pinot-contrib/pinot-docs/pull/109)).
Not sure why it hasn't been updated on documentatio
mcvsubbu commented on code in PR #9289:
URL: https://github.com/apache/pinot/pull/9289#discussion_r957635296
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java:
##
@@ -1074,6 +1077,15 @@ IdealState ensureAllParti
saurabhd336 opened a new pull request, #9294:
URL: https://github.com/apache/pinot/pull/9294
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`
walterddr commented on issue #9204:
URL: https://github.com/apache/pinot/issues/9204#issuecomment-1230635716
updated the descriptions, thanks @Jackie-Jiang , i think we can actually
support these out of MV columns. see: previous issue
https://github.com/apache/pinot/issues/6083.
I th
sajjad-moradi commented on code in PR #9289:
URL: https://github.com/apache/pinot/pull/9289#discussion_r957621762
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotRealtimeTableResource.java:
##
@@ -77,14 +77,29 @@ public Response pauseConsumption(
sajjad-moradi commented on code in PR #9289:
URL: https://github.com/apache/pinot/pull/9289#discussion_r957614663
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotRealtimeTableResource.java:
##
@@ -77,14 +77,29 @@ public Response pauseConsumption(
sajjad-moradi opened a new pull request, #9293:
URL: https://github.com/apache/pinot/pull/9293
AccessType for a couple of Schema REST endpoints were configured
incorrectly. This PR fixes those issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, p
mcvsubbu commented on code in PR #9289:
URL: https://github.com/apache/pinot/pull/9289#discussion_r957559540
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotRealtimeTableResource.java:
##
@@ -77,14 +77,29 @@ public Response pauseConsumption(
@
SabrinaZhaozyf commented on PR #9114:
URL: https://github.com/apache/pinot/pull/9114#issuecomment-1230513674
@npawar Changes to user doc has been merged here
(https://github.com/pinot-contrib/pinot-docs/pull/109). Not sure why it hasn't
been updated on documentation yet.
--
This is an au
mcvsubbu commented on code in PR #9289:
URL: https://github.com/apache/pinot/pull/9289#discussion_r957479004
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotRealtimeTableResource.java:
##
@@ -77,14 +77,29 @@ public Response pauseConsumption(
@
walterddr opened a new issue, #9292:
URL: https://github.com/apache/pinot/issues/9292
Currently, if a query is run with the ORDER BY clause. data is
- in selection only, data is sorted and trimmed to the limit
- in aggregation/group-by/distinct, data is unsorted but trimmed to the
lim
walterddr commented on code in PR #8927:
URL: https://github.com/apache/pinot/pull/8927#discussion_r957435868
##
pinot-core/src/main/java/org/apache/pinot/core/query/selection/SelectionOperatorUtils.java:
##
@@ -227,16 +229,42 @@ public static void
mergeWithOrdering(PriorityQue
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 15b16e8f10 Improve server query cancellation
siddharthteotia merged PR #9286:
URL: https://github.com/apache/pinot/pull/9286
--
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...@pino
44 matches
Mail list logo
