gortiz commented on code in PR #10977:
URL: https://github.com/apache/pinot/pull/10977#discussion_r1244761724
##
pinot-core/src/main/java/org/apache/pinot/core/util/trace/BuiltInTracer.java:
##
@@ -50,6 +50,15 @@ public void close() {
}
TraceContext.logTime(operato
robertzych commented on code in PR #10463:
URL: https://github.com/apache/pinot/pull/10463#discussion_r1244647288
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeTableDataManager.java:
##
@@ -489,14 +489,16 @@ static void setDefaultTimeValueIfInva
codecov-commenter commented on PR #10990:
URL: https://github.com/apache/pinot/pull/10990#issuecomment-1610632503
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/10990?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
abhioncbr commented on code in PR #10990:
URL: https://github.com/apache/pinot/pull/10990#discussion_r1244585380
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/segment/creator/impl/SegmentColumnarIndexCreatorTest.java:
##
@@ -174,21 +202,52 @@ private static
abhioncbr opened a new pull request, #10990:
URL: https://github.com/apache/pinot/pull/10990
Previously, we skipped to set the value for min/max if the value was
invalid. We were determining invalid values based on the following checks
- If value contains leading or trailing whitespace c
itschrispeck opened a new pull request, #10989:
URL: https://github.com/apache/pinot/pull/10989
Issue described in: https://github.com/apache/pinot/issues/10976
Tested the cherry-picked commit on an internal cluster, verified via logs
that `_luceneRealtimeReaders` no longer constantly
Jackie-Jiang commented on PR #10985:
URL: https://github.com/apache/pinot/pull/10985#issuecomment-1610421570
cc @vvivekiyer @somandal This should simplify the aggregation function
creation on V2 engine
--
This is an automated message from the Apache Git Service.
To respond to the message,
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 04bdb9a397 Enhance early terminate for combine op
Jackie-Jiang merged PR #10988:
URL: https://github.com/apache/pinot/pull/10988
--
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 commented on code in PR #10969:
URL: https://github.com/apache/pinot/pull/10969#discussion_r1244528990
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotInstanceRestletResource.java:
##
@@ -390,4 +391,33 @@ public SuccessResponse updat
Jackie-Jiang commented on code in PR #10926:
URL: https://github.com/apache/pinot/pull/10926#discussion_r1244507824
##
pinot-core/src/test/java/org/apache/pinot/core/common/ObjectSerDeUtilsTest.java:
##
@@ -374,4 +378,38 @@ public void testDouble2LongMap() {
assertEquals(
codecov-commenter commented on PR #10988:
URL: https://github.com/apache/pinot/pull/10988#issuecomment-1610407375
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/10988?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
abhioncbr commented on code in PR #10891:
URL: https://github.com/apache/pinot/pull/10891#discussion_r1244511327
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/creator/impl/SegmentColumnarIndexCreator.java:
##
@@ -570,12 +570,14 @@ public static void
Jackie-Jiang commented on code in PR #10463:
URL: https://github.com/apache/pinot/pull/10463#discussion_r1244504059
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeTableDataManager.java:
##
@@ -489,14 +489,16 @@ static void setDefaultTimeValueIfIn
codecov-commenter commented on PR #10985:
URL: https://github.com/apache/pinot/pull/10985#issuecomment-1610386681
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/10985?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
codecov-commenter commented on PR #10987:
URL: https://github.com/apache/pinot/pull/10987#issuecomment-1610382874
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/10987?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
walterddr closed pull request #10984: [bugfix] check exception before create
group-by result block
URL: https://github.com/apache/pinot/pull/10984
--
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
walterddr commented on PR #10984:
URL: https://github.com/apache/pinot/pull/10984#issuecomment-1610380414
+1
--
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,
Jackie-Jiang commented on PR #10984:
URL: https://github.com/apache/pinot/pull/10984#issuecomment-1610379488
We don't really need to merge the exceptions since we don't return the
result anyway. Early termination should be more desired. See #10988
--
This is an automated message from the
Jackie-Jiang opened a new pull request, #10988:
URL: https://github.com/apache/pinot/pull/10988
- When exception happens during query execution, stop processing the
remaining segments for all threads
- (bugfix) Remove the member variable override in
`BaseSingleBlockCombineOperator` which
swaminathanmanish commented on PR #10987:
URL: https://github.com/apache/pinot/pull/10987#issuecomment-1610354443
cc @snleee - Please 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
URL above to go to
swaminathanmanish opened a new pull request, #10987:
URL: https://github.com/apache/pinot/pull/10987
**Problem**:
Currently, we dont have size based segment creation for batch ingestion that
uses SegmentProcessorFramework. With maxNumRecordsPerSegment, we cannot create
segment of desire
wirybeaver commented on issue #10981:
URL: https://github.com/apache/pinot/issues/10981#issuecomment-1610353025
All segments are in Consuming State, i.e., mutable segment
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
walterddr commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r1244468924
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
} c
wirybeaver commented on issue #10981:
URL: https://github.com/apache/pinot/issues/10981#issuecomment-1610334868
```
SELECT *
FROM rta_odin_analytics_staging_audit_v6
where JSON_MATCH(direct_tags, '"$.root_workflow[0].tag_value" = ''foo''')
```
works
--
This is an automated m
wirybeaver commented on issue #10981:
URL: https://github.com/apache/pinot/issues/10981#issuecomment-1610332723
The pinot field spec:
{
"name": "direct_tags",
"dataType": "STRING",
"maxLength": 102400
}
--
This is an automated message from the Apach
shwin opened a new issue, #10986:
URL: https://github.com/apache/pinot/issues/10986
we're seeing something strange with timestamp indices and segment pruning.
```
We have a timestamp index like this:
{
"name": "blockTimestamp",
"encodingType": "DICTIONARY",
andimiller commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r1244438998
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
}
Jackie-Jiang opened a new pull request, #10985:
URL: https://github.com/apache/pinot/pull/10985
Context:
- When introducing the first version of `DISTINCT`, we treat it as
aggregation. Then we found a lot of limitations because of that. To solve the
limitations we implemented a dedicate
andimiller commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r124492
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
}
andimiller commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r124492
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
}
andimiller commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r1244438998
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
}
codecov-commenter commented on PR #10984:
URL: https://github.com/apache/pinot/pull/10984#issuecomment-1610304258
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/10984?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
robertzych commented on code in PR #10463:
URL: https://github.com/apache/pinot/pull/10463#discussion_r1244432499
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeTableDataManager.java:
##
@@ -489,14 +489,16 @@ static void setDefaultTimeValueIfInva
chenboat commented on issue #10981:
URL: https://github.com/apache/pinot/issues/10981#issuecomment-1610280711
cc @Jackie-Jiang @mayankshriv
--
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 sp
walterddr commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r1244407923
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
} c
walterddr commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r1244407923
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
} c
walterddr commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r1244407923
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
} c
walterddr commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r1244407923
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
} c
andimiller commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r1244392095
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
}
walterddr opened a new pull request, #10984:
URL: https://github.com/apache/pinot/pull/10984
fix NPE when indexTable is not created but exception is thrown.
--
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
codecov-commenter commented on PR #10983:
URL: https://github.com/apache/pinot/pull/10983#issuecomment-1610245244
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/10983?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
codecov-commenter commented on PR #10979:
URL: https://github.com/apache/pinot/pull/10979#issuecomment-1610240202
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/10979?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
walterddr commented on code in PR #10937:
URL: https://github.com/apache/pinot/pull/10937#discussion_r1244338599
##
pinot-broker/src/main/java/org/apache/pinot/broker/api/resources/InstanceResource.java:
##
@@ -0,0 +1,69 @@
+/**
+ * Licensed to the Apache Software Foundation (AS
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 b001c40182 [multistage][feature] pipeline breaker on broker reduce
(#10971)
add 238629c41c Support NULL for multi
Jackie-Jiang merged PR #10938:
URL: https://github.com/apache/pinot/pull/10938
--
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
klsince commented on code in PR #10977:
URL: https://github.com/apache/pinot/pull/10977#discussion_r1244334712
##
pinot-core/src/main/java/org/apache/pinot/core/util/trace/BuiltInTracer.java:
##
@@ -50,6 +50,15 @@ public void close() {
}
TraceContext.logTime(operat
Jackie-Jiang commented on code in PR #10905:
URL: https://github.com/apache/pinot/pull/10905#discussion_r1244331242
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/startree/v2/builder/MultipleTreesBuilder.java:
##
@@ -83,7 +86,11 @@ public MultipleTreesBuilde
klsince commented on code in PR #10977:
URL: https://github.com/apache/pinot/pull/10977#discussion_r1244334712
##
pinot-core/src/main/java/org/apache/pinot/core/util/trace/BuiltInTracer.java:
##
@@ -50,6 +50,15 @@ public void close() {
}
TraceContext.logTime(operat
walterddr merged PR #10971:
URL: https://github.com/apache/pinot/pull/10971
--
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.ap
This is an automated email from the ASF dual-hosted git repository.
rongr 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 3b07def331 use MultiStageRequestIdGenerator in v1
klsince merged PR #10977:
URL: https://github.com/apache/pinot/pull/10977
--
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
This is an automated email from the ASF dual-hosted git repository.
rongr pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from fc9f56b07b allow to add custom context into query trace (#10977)
add b001c40182 [multistage][feature] pipeline brea
walterddr closed issue #10131: [multistage] Broker busy waiting
URL: https://github.com/apache/pinot/issues/10131
--
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 unsubscrib
This is an automated email from the ASF dual-hosted git repository.
xbli pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 3b07def331 use MultiStageRequestIdGenerator in v1 engine (#10966)
add fc9f56b07b allow to add custom context into qu
walterddr merged PR #10966:
URL: https://github.com/apache/pinot/pull/10966
--
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.ap
walterddr commented on code in PR #10977:
URL: https://github.com/apache/pinot/pull/10977#discussion_r1244329566
##
pinot-core/src/main/java/org/apache/pinot/core/util/trace/BuiltInTracer.java:
##
@@ -50,6 +50,15 @@ public void close() {
}
TraceContext.logTime(oper
walterddr commented on code in PR #10982:
URL: https://github.com/apache/pinot/pull/10982#discussion_r1244327305
##
pinot-segment-local/src/test/java/org/apache/pinot/segment/local/utils/TableConfigUtilsTest.java:
##
@@ -1755,6 +1755,14 @@ public void testTaskConfig() {
} c
walterddr commented on code in PR #10983:
URL: https://github.com/apache/pinot/pull/10983#discussion_r1244326440
##
pinot-spi/src/test/java/org/apache/pinot/spi/stream/StreamDataDecoderImplTest.java:
##
@@ -41,7 +41,7 @@ public void testDecodeValueOnly()
messageDecoder.init
Jackie-Jiang commented on code in PR #10463:
URL: https://github.com/apache/pinot/pull/10463#discussion_r1244318587
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeTableDataManager.java:
##
@@ -489,14 +489,16 @@ static void setDefaultTimeValueIfIn
summerhasama-stripe opened a new pull request, #10983:
URL: https://github.com/apache/pinot/pull/10983
We saw a lot of logs for the FreshnessBasedConsumptionStatusChecker that
show the servers are not aware of the freshness of incoming events
Through some debugging, we determined the
robertzych commented on code in PR #10463:
URL: https://github.com/apache/pinot/pull/10463#discussion_r1244251665
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeTableDataManager.java:
##
@@ -489,14 +489,16 @@ static void setDefaultTimeValueIfInva
ksnijjer commented on PR #10982:
URL: https://github.com/apache/pinot/pull/10982#issuecomment-1610041218
@mayankshriv FYI
--
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
codecov-commenter commented on PR #10982:
URL: https://github.com/apache/pinot/pull/10982#issuecomment-1610021184
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/10982?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
walterddr commented on code in PR #10873:
URL: https://github.com/apache/pinot/pull/10873#discussion_r1244103752
##
pinot-query-planner/src/main/java/org/apache/pinot/query/QueryEnvironment.java:
##
@@ -208,6 +210,21 @@ public String explainQuery(String sqlQuery) {
return e
andimiller opened a new pull request, #10982:
URL: https://github.com/apache/pinot/pull/10982
The only thing blocking these being used was the table validator in
`TableConfigUtils` so I've updated it.
--
This is an automated message from the Apache Git Service.
To respond to the message,
walterddr commented on code in PR #10873:
URL: https://github.com/apache/pinot/pull/10873#discussion_r1244098059
##
pinot-query-planner/src/main/java/org/apache/pinot/query/QueryEnvironment.java:
##
@@ -208,6 +210,24 @@ public String explainQuery(String sqlQuery) {
return e
gortiz commented on PR #10953:
URL: https://github.com/apache/pinot/pull/10953#issuecomment-1609272970
Can we merge 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 go to the specific comment.
To
tsegismont commented on PR #10326:
URL: https://github.com/apache/pinot/pull/10326#issuecomment-1609021564
Thank you @Jackie-Jiang
--
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 co
jtao15 commented on code in PR #10964:
URL: https://github.com/apache/pinot/pull/10964#discussion_r1243289538
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/lineage/DefaultLineageManager.java:
##
@@ -105,6 +107,11 @@ public void updateLineageForRetentio
70 matches
Mail list logo