dependabot[bot] commented on PR #15152:
URL: https://github.com/apache/pinot/pull/15152#issuecomment-2691422641
OK, I won't notify you again about this release, but will get in touch when
a new version is available. You can also ignore all major, minor, or patch
releases for a dependency by
Jackie-Jiang merged PR #15157:
URL: https://github.com/apache/pinot/pull/15157
--
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
codecov-commenter commented on PR #15158:
URL: https://github.com/apache/pinot/pull/15158#issuecomment-2691900738
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15158?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
xiangfu0 closed pull request #15117: [draft]Add a new distinctCountGlobal
function
URL: https://github.com/apache/pinot/pull/15117
--
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 commen
9aman commented on code in PR #15155:
URL: https://github.com/apache/pinot/pull/15155#discussion_r1975926133
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/SegmentDeletionManager.java:
##
@@ -349,23 +350,36 @@ public void
removeAgedDeletedSegments(Lead
Jackie-Jiang opened a new pull request, #15158:
URL: https://github.com/apache/pinot/pull/15158
Currently aggregation ser/de is handled by `ObjectSerDeUtils`. It has the
following drawbacks:
- Ser/de is not self-contained in the `AggregationFunction`
- Hard to maintain backward compati
ankitsultana commented on PR #15151:
URL: https://github.com/apache/pinot/pull/15151#issuecomment-2691796187
@vrajat : any PEP or design doc for 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
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 9c7aef745d Upgrade Pulsar to 4.0.3 (#15157)
9c7ae
Jackie-Jiang closed issue #14543: Ingest: update Pulsar to 4.0 LTS
URL: https://github.com/apache/pinot/issues/14543
--
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
deemoliu commented on code in PR #15107:
URL: https://github.com/apache/pinot/pull/15107#discussion_r1976093775
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/merger/PartialUpsertColumnarMerger.java:
##
@@ -74,7 +75,10 @@ public void merge(LazyRow pre
codecov-commenter commented on PR #15157:
URL: https://github.com/apache/pinot/pull/15157#issuecomment-2691544006
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15157?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
Jackie-Jiang commented on code in PR #15143:
URL: https://github.com/apache/pinot/pull/15143#discussion_r1975980844
##
pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java:
##
@@ -248,6 +248,9 @@ public static class Instance {
public static final String
J-HowHuang commented on code in PR #15138:
URL: https://github.com/apache/pinot/pull/15138#discussion_r1975982749
##
pinot-core/src/test/java/org/apache/pinot/queries/ExplainPlanQueriesTest.java:
##
@@ -2301,13 +2298,12 @@ public void
testSelectAggregateUsingFilterIndexVerbose(
Jackie-Jiang opened a new pull request, #15157:
URL: https://github.com/apache/pinot/pull/15157
Close #14543
--
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 code in PR #15149:
URL: https://github.com/apache/pinot/pull/15149#discussion_r1975948391
##
pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountHighCardinalityAggregationFunction.java:
##
@@ -0,0 +1,253 @@
+/**
+ * Li
ashishjayamohan commented on code in PR #15149:
URL: https://github.com/apache/pinot/pull/15149#discussion_r1975951176
##
pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountHighCardinalityAggregationFunction.java:
##
@@ -0,0 +1,253 @@
+/**
+ *
Jackie-Jiang commented on code in PR #15149:
URL: https://github.com/apache/pinot/pull/15149#discussion_r1975940592
##
pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/DistinctCountHighCardinalityAggregationFunction.java:
##
@@ -0,0 +1,253 @@
+/**
+ * Li
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 fdd6a1aa86 Fix issue of trailing / in the path for gcs (#15155)
add 941c93daf8 Bump com.microsoft.azure:msal4j fro
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch
dependabot/maven/com.microsoft.azure-msal4j-1.19.1
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 31fa5c0e68 Bump com.microsoft.azure:msal4j from 1.19.0 to 1.19.1
The revi
Jackie-Jiang merged PR #15153:
URL: https://github.com/apache/pinot/pull/15153
--
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, #15156:
URL: https://github.com/apache/pinot/pull/15156
(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,
xiangfu0 merged PR #15155:
URL: https://github.com/apache/pinot/pull/15155
--
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
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 51e7b84bd2 Bump software.amazon.awssdk:bom from 2.30.28 to 2.30.30
(#15154)
add fdd6a1aa86 Fix issue of trailing
xiangfu0 commented on code in PR #15155:
URL: https://github.com/apache/pinot/pull/15155#discussion_r1975927119
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/SegmentDeletionManager.java:
##
@@ -349,23 +350,36 @@ public void
removeAgedDeletedSegments(L
albertobastos commented on code in PR #15143:
URL: https://github.com/apache/pinot/pull/15143#discussion_r1975905037
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/QueryRunner.java:
##
@@ -174,9 +174,14 @@ public void init(PinotConfiguration config,
Instanc
albertobastos commented on code in PR #15143:
URL: https://github.com/apache/pinot/pull/15143#discussion_r1975907262
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/QueryRunner.java:
##
@@ -177,6 +178,12 @@ public void init(PinotConfiguration config,
Instanc
albertobastos commented on code in PR #15143:
URL: https://github.com/apache/pinot/pull/15143#discussion_r1975906078
##
pinot-spi/src/main/java/org/apache/pinot/spi/executor/ExecutorServiceUtils.java:
##
@@ -132,4 +133,14 @@ public static void close(ExecutorService executorServi
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975584205
##
pinot-spi/src/main/java/org/apache/pinot/spi/exception/QueryErrorMessage.java:
##
@@ -0,0 +1,94 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
gortiz commented on PR #15037:
URL: https://github.com/apache/pinot/pull/15037#issuecomment-2691010316
I've pushed several changes to fix most of the comments from @yashmayya. I
think I also commented on the open discussions
--
This is an automated message from the Apache Git Service.
To
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975647555
##
pinot-spi/src/main/java/org/apache/pinot/spi/exception/QueryException.java:
##
@@ -0,0 +1,62 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975580497
##
pinot-spi/src/main/java/org/apache/pinot/spi/exception/EarlyTerminationException.java:
##
@@ -22,17 +22,17 @@
* The {@code EarlyTerminationException} can be thrown
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975628058
##
pinot-core/src/main/java/org/apache/pinot/core/operator/streaming/StreamingInstanceResponseOperator.java:
##
@@ -89,11 +89,12 @@ protected InstanceResponseBlock getNe
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975628058
##
pinot-core/src/main/java/org/apache/pinot/core/operator/streaming/StreamingInstanceResponseOperator.java:
##
@@ -89,11 +89,12 @@ protected InstanceResponseBlock getNe
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975625657
##
pinot-core/src/main/java/org/apache/pinot/core/operator/combine/BaseSingleBlockCombineOperator.java:
##
@@ -112,11 +113,17 @@ protected void processSegments() {
@O
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975592710
##
pinot-spi/src/main/java/org/apache/pinot/spi/exception/QueryErrorMessage.java:
##
@@ -0,0 +1,94 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975623209
##
pinot-core/src/main/java/org/apache/pinot/core/operator/combine/BaseSingleBlockCombineOperator.java:
##
@@ -65,7 +66,7 @@ protected BaseResultsBlock getNextBlock() {
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975619798
##
pinot-core/src/main/java/org/apache/pinot/core/operator/combine/BaseCombineOperator.java:
##
@@ -190,16 +191,21 @@ protected static RuntimeException
wrapOperatorExce
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975576427
##
pinot-spi/src/main/java/org/apache/pinot/spi/exception/QueryErrorCode.java:
##
@@ -0,0 +1,141 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+
swaminathanmanish commented on code in PR #15155:
URL: https://github.com/apache/pinot/pull/15155#discussion_r1975482332
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/SegmentDeletionManager.java:
##
@@ -349,23 +350,36 @@ public void
removeAgedDeletedS
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975563360
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/DataBlock.java:
##
@@ -41,10 +42,15 @@ public interface DataBlock {
int getNumberOfColumns();
+ @
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975549617
##
pinot-common/src/main/java/org/apache/pinot/common/response/broker/BrokerResponseNative.java:
##
@@ -59,16 +59,16 @@
public class BrokerResponseNative implements Bro
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975562169
##
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java:
##
@@ -726,18 +722,18 @@ protected BrokerResponse doHandle
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975559186
##
pinot-common/src/main/java/org/apache/pinot/common/response/broker/BrokerResponseNative.java:
##
@@ -59,16 +59,16 @@
public class BrokerResponseNative implements Bro
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975547455
##
pinot-common/src/main/java/org/apache/pinot/common/function/FunctionInvoker.java:
##
@@ -112,7 +113,11 @@ public void convertTypes(Object[] arguments) {
PinotD
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975530174
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/DataBlockUtils.java:
##
@@ -97,18 +98,22 @@ private DataBlockUtils() {
public static MetadataBlock g
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975539639
##
pinot-common/src/main/java/org/apache/pinot/common/datatable/DataTable.java:
##
@@ -36,10 +37,15 @@
*/
public interface DataTable {
+ @Deprecated
void addExc
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975539639
##
pinot-common/src/main/java/org/apache/pinot/common/datatable/DataTable.java:
##
@@ -36,10 +37,15 @@
*/
public interface DataTable {
+ @Deprecated
void addExc
yashmayya merged PR #15154:
URL: https://github.com/apache/pinot/pull/15154
--
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
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975524355
##
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/MultiStageBrokerRequestHandler.java:
##
@@ -222,16 +222,16 @@ protected BrokerResponse handleRequest
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975514955
##
pinot-common/src/main/java/org/apache/pinot/common/response/BrokerResponse.java:
##
@@ -76,11 +76,11 @@ default void toOutputStream(OutputStream outputStream)
/**
gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975514955
##
pinot-common/src/main/java/org/apache/pinot/common/response/BrokerResponse.java:
##
@@ -76,11 +76,11 @@ default void toOutputStream(OutputStream outputStream)
/**
codecov-commenter commented on PR #15155:
URL: https://github.com/apache/pinot/pull/15155#issuecomment-2690753120
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/15155?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.
yashmayya pushed a change to branch
dependabot/maven/software.amazon.awssdk-bom-2.30.30
in repository https://gitbox.apache.org/repos/asf/pinot.git
was d95ea1f023 Bump software.amazon.awssdk:bom from 2.30.28 to 2.30.30
Th
This is an automated email from the ASF dual-hosted git repository.
yashmayya pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from fd3b079615 Refactor DispatchableSubPlan to store fragments by stage id
(#15135)
add 51e7b84bd2 Bump software.a
This is an automated email from the ASF dual-hosted git repository.
gortiz 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 fd3b079615 Refactor DispatchableSubPlan to store
gortiz merged PR #15135:
URL: https://github.com/apache/pinot/pull/15135
--
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.apach
yashmayya commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1975177226
##
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/MultiStageBrokerRequestHandler.java:
##
@@ -368,9 +364,10 @@ protected BrokerResponse handleReque
gortiz commented on code in PR #15143:
URL: https://github.com/apache/pinot/pull/15143#discussion_r1975455876
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/QueryRunner.java:
##
@@ -177,6 +178,12 @@ public void init(PinotConfiguration config,
InstanceDataMa
gortiz commented on code in PR #15143:
URL: https://github.com/apache/pinot/pull/15143#discussion_r1975452181
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/plan/server/ServerPlanRequestUtils.java:
##
@@ -106,7 +106,7 @@ public static OpChain
compileLeafSta
9aman opened a new pull request, #15155:
URL: https://github.com/apache/pinot/pull/15155
## Issue
The GcsPinotFS returns directories names with training "/" unlike S3PinotFS
and LocalPinotFS. This leads to issue in deletion of segments by the retention
manager as it expects sanitized
tibrewalpratik17 commented on PR #15107:
URL: https://github.com/apache/pinot/pull/15107#issuecomment-2690667837
LGTM!
cc @Jackie-Jiang @klsince in case you have any comments!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
albertobastos commented on code in PR #15143:
URL: https://github.com/apache/pinot/pull/15143#discussion_r1975237338
##
pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java:
##
@@ -1065,6 +1065,9 @@ public static class SegmentCompletionProtocol {
public
dependabot[bot] opened a new pull request, #15152:
URL: https://github.com/apache/pinot/pull/15152
Bumps `pulsar.version` from 3.3.1 to 4.0.3.
Updates `org.apache.pulsar:pulsar-client` from 3.3.1 to 4.0.3
Release notes
Sourced from https://github.com/apache/pulsar/releases";>org.
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.30.29
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 0860722128 Bump software.amazon.awssdk:bom from 2.30.28 to 2.30.29
T
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.30.30
in repository https://gitbox.apache.org/repos/asf/pinot.git
at d95ea1f023 Bump software.amazon.awssdk:bom from 2.30.28 to 2.30.30
N
dependabot[bot] opened a new pull request, #15154:
URL: https://github.com/apache/pinot/pull/15154
Bumps software.amazon.awssdk:bom from 2.30.28 to 2.30.30.
Most Recent Ignore Conditions Applied to This Pull Request
| Dependency Name | Ignore Conditions |
| --- | --- |
dependabot[bot] commented on PR #15145:
URL: https://github.com/apache/pinot/pull/15145#issuecomment-2690370597
Superseded by #15154.
--
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
dependabot[bot] closed pull request #15145: Bump software.amazon.awssdk:bom
from 2.30.28 to 2.30.29
URL: https://github.com/apache/pinot/pull/15145
--
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
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/com.microsoft.azure-msal4j-1.19.1
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 31fa5c0e68 Bump com.microsoft.azure:msal4j from 1.19.0 to 1.19.1
No n
dependabot[bot] opened a new pull request, #15153:
URL: https://github.com/apache/pinot/pull/15153
Bumps
[com.microsoft.azure:msal4j](https://github.com/AzureAD/microsoft-authentication-library-for-java)
from 1.19.0 to 1.19.1.
Changelog
Sourced from https://github.com/AzureAD/micr
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch dependabot/maven/pulsar.version-4.0.3
in repository https://gitbox.apache.org/repos/asf/pinot.git
at eadfcd65e7 Bump pulsar.version from 3.3.1 to 4.0.3
No new revisions were added by t
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 ec87186017 Replace More String.format() References With Concatenation
(#15148)
add a0037d3be8 Nit: Fix Error Mess
albertobastos commented on code in PR #15143:
URL: https://github.com/apache/pinot/pull/15143#discussion_r1975012300
##
pinot-core/src/main/java/org/apache/pinot/core/query/executor/MdcQueryExecutor.java:
##
@@ -0,0 +1,108 @@
+/**
+ * Licensed to the Apache Software Foundation (
73 matches
Mail list logo
