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 ffc72b1019 Add @JsonProperty annotation to LeadCo
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 d04e361ef1 Improve dimension table handing (#1396
deepthi912 commented on code in PR #13950:
URL: https://github.com/apache/pinot/pull/13950#discussion_r1751059419
##
pinot-controller/src/test/java/org/apache/pinot/controller/util/JsonSerializerTest.java:
##
@@ -0,0 +1,187 @@
+/**
+ * Licensed to the Apache Software Foundation
deepthi912 commented on code in PR #13950:
URL: https://github.com/apache/pinot/pull/13950#discussion_r1751059419
##
pinot-controller/src/test/java/org/apache/pinot/controller/util/JsonSerializerTest.java:
##
@@ -0,0 +1,187 @@
+/**
+ * Licensed to the Apache Software Foundation
Jackie-Jiang commented on code in PR #13967:
URL: https://github.com/apache/pinot/pull/13967#discussion_r1751047670
##
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/LookupTransformFunction.java:
##
@@ -219,8 +218,7 @@ private void lookup(ValueBlock v
klsince commented on code in PR #13967:
URL: https://github.com/apache/pinot/pull/13967#discussion_r1751039460
##
pinot-core/src/main/java/org/apache/pinot/core/operator/transform/function/LookupTransformFunction.java:
##
@@ -219,8 +218,7 @@ private void lookup(ValueBlock valueB
Jackie-Jiang merged PR #13965:
URL: https://github.com/apache/pinot/pull/13965
--
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 opened a new pull request, #13966:
URL: https://github.com/apache/pinot/pull/13966
On top of #13965
- Add `lookup` join strategy as a hint
- Improve dimension table performance
TODO: Improve planner rules to not push down expression/filter into the
dimension ta
Jackie-Jiang opened a new pull request, #13965:
URL: https://github.com/apache/pinot/pull/13965
Also cleans up the rule set for readability
--
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 spe
codecov-commenter commented on PR #13964:
URL: https://github.com/apache/pinot/pull/13964#issuecomment-2339100551
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13964?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
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 b828280882 Part-1: Pinot Timeseries Engine SPI (#
Jackie-Jiang merged PR #13885:
URL: https://github.com/apache/pinot/pull/13885
--
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 #13963:
URL: https://github.com/apache/pinot/pull/13963
--
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
tibrewalpratik17 opened a new pull request, #13964:
URL: https://github.com/apache/pinot/pull/13964
(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 unsubscr
praveenc7 commented on code in PR #13952:
URL: https://github.com/apache/pinot/pull/13952#discussion_r1750889890
##
pinot-broker/src/main/java/org/apache/pinot/broker/routing/instanceselector/ReplicaGroupInstanceSelector.java:
##
@@ -152,21 +155,28 @@ private Pair, Map>
selectS
codecov-commenter commented on PR #13963:
URL: https://github.com/apache/pinot/pull/13963#issuecomment-2339007409
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13963?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
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.27.21
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 6c7ba52b48 Bump software.amazon.awssdk:bom from 2.27.20 to 2.27.21
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 520eb991ae Rename dangling occurrences of StageNode to PlanNode
(#13962)
add 05d7fd85cf Bump software.amazon.awss
Jackie-Jiang merged PR #13962:
URL: https://github.com/apache/pinot/pull/13962
--
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 opened a new pull request, #13963:
URL: https://github.com/apache/pinot/pull/13963
Address remaining comments for #13863
--
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
Jackie-Jiang commented on issue #13833:
URL: https://github.com/apache/pinot/issues/13833#issuecomment-2338854162
> There is no way to use ST index in this case but it also prevents inv
index from being created/used.
@mayankshriv Inverted index is being created/used. What is missing i
Jackie-Jiang commented on code in PR #13950:
URL: https://github.com/apache/pinot/pull/13950#discussion_r1750761802
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotLeadControllerRestletResource.java:
##
@@ -211,12 +212,16 @@ public List getTableN
Jackie-Jiang commented on issue #13472:
URL: https://github.com/apache/pinot/issues/13472#issuecomment-2338830528
Helix has released
[1.4.1](https://helix.apache.org/1.4.1-docs/releasenotes/release-1.4.1.html).
We should directly bump to the latest version
--
This is an automated message
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/log4j.version-2.24.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was cf2ab3a264 Bump log4j.version from 2.23.1 to 2.24.0
The revisions that were on this bra
Jackie-Jiang commented on code in PR #13953:
URL: https://github.com/apache/pinot/pull/13953#discussion_r1750726919
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java:
##
@@ -3722,8 +3722,12 @@ public String startReplaceSegmen
This is an automated email from the ASF dual-hosted git repository.
tingchen 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 d73aa1ee31 Add Segment Relocator Rebalance Conf
chenboat merged PR #13863:
URL: https://github.com/apache/pinot/pull/13863
--
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
tibrewalpratik17 commented on code in PR #13914:
URL: https://github.com/apache/pinot/pull/13914#discussion_r1750640540
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java:
##
@@ -544,9 +544,7 @@ public void addSegme
anandheritage commented on code in PR #13954:
URL: https://github.com/apache/pinot/pull/13954#discussion_r1750623794
##
pinot-common/src/main/java/org/apache/pinot/common/utils/helix/HelixHelper.java:
##
@@ -542,6 +549,12 @@ public static List
getInstancesWithTag(List instanceC
mayankshriv commented on code in PR #13954:
URL: https://github.com/apache/pinot/pull/13954#discussion_r1750606115
##
pinot-common/src/main/java/org/apache/pinot/common/utils/helix/HelixHelper.java:
##
@@ -542,6 +549,12 @@ public static List
getInstancesWithTag(List instanceCon
This is an automated email from the ASF dual-hosted git repository.
xbli 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 a9e4e5289f [Backfill] flink based upsert table back
gortiz commented on code in PR #13303:
URL: https://github.com/apache/pinot/pull/13303#discussion_r1750400466
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/DataBlockUtils.java:
##
@@ -18,20 +18,48 @@
*/
package org.apache.pinot.common.datablock;
+import co
codecov-commenter commented on PR #13962:
URL: https://github.com/apache/pinot/pull/13962#issuecomment-2338290038
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13962?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1750381217
##
pinot-common/src/main/java/org/apache/pinot/common/utils/DatabaseUtils.java:
##
@@ -32,6 +32,10 @@ public class DatabaseUtils {
private DatabaseUtils() {
}
+
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1750350125
##
pinot-core/src/main/java/org/apache/pinot/core/operator/ProjectionOperator.java:
##
@@ -100,6 +100,17 @@ public String toExplainString() {
return stringBuilder.a
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1750345793
##
pinot-core/src/main/java/org/apache/pinot/core/operator/query/AggregationOperator.java:
##
@@ -100,4 +103,21 @@ public String toExplainString() {
return string
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1750325812
##
pinot-core/src/main/java/org/apache/pinot/core/operator/query/FilteredGroupByOperator.java:
##
@@ -230,4 +232,26 @@ public String toExplainString() {
return st
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1750317381
##
pinot-common/src/main/proto/plan.proto:
##
@@ -193,3 +194,29 @@ message WindowNode {
int32 upperBound = 6;
repeated Literal constants = 7;
}
+
+message Explain
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1750308034
##
pinot-core/src/main/java/org/apache/pinot/core/plan/PinotExplainedRelNode.java:
##
@@ -0,0 +1,146 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under o
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1750300488
##
pinot-core/src/main/java/org/apache/pinot/core/plan/PinotExplainedRelNode.java:
##
@@ -0,0 +1,146 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under o
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1750217340
##
pinot-core/src/main/java/org/apache/pinot/core/query/request/context/QueryContext.java:
##
@@ -85,7 +85,7 @@ public class QueryContext {
private final int _offset;
gortiz commented on code in PR #13733:
URL: https://github.com/apache/pinot/pull/13733#discussion_r1750212242
##
pinot-core/src/main/java/org/apache/pinot/core/query/request/context/QueryContext.java:
##
@@ -528,7 +538,16 @@ public Builder
setExpressionOverrideHints(Map
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/software.amazon.awssdk-bom-2.27.21
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 6c7ba52b48 Bump software.amazon.awssdk:bom from 2.27.20 to 2.27.21
N
dependabot[bot] opened a new pull request, #13961:
URL: https://github.com/apache/pinot/pull/13961
Bumps software.amazon.awssdk:bom from 2.27.20 to 2.27.21.
[