Jackie-Jiang commented on code in PR #12886:
URL: https://github.com/apache/pinot/pull/12886#discussion_r1577304282
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/data/manager/TableDataManager.java:
##
@@ -76,22 +79,52 @@ void init(InstanceDataManagerConfig
Jackie-Jiang commented on code in PR #12886:
URL: https://github.com/apache/pinot/pull/12886#discussion_r1577299251
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeTableDataManager.java:
##
@@ -570,165 +563,57 @@ private void
Jackie-Jiang commented on code in PR #12886:
URL: https://github.com/apache/pinot/pull/12886#discussion_r1577266557
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeSegmentDataManager.java:
##
@@ -752,14 +752,20 @@ public void run() {
Jackie-Jiang commented on code in PR #12886:
URL: https://github.com/apache/pinot/pull/12886#discussion_r1577264671
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java:
##
@@ -608,172 +721,153 @@ protected SegmentDataManager
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 bc9e8ee541 Upgrade Pulsar to 3.2.2 (#12967)
KKcorps merged PR #12967:
URL: https://github.com/apache/pinot/pull/12967
--
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:
KKcorps closed issue #12713: Ingest from Pulsar: relying on/ compatible to EOL
version of Pulsar
URL: https://github.com/apache/pinot/issues/12713
--
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
codecov-commenter commented on PR #12993:
URL: https://github.com/apache/pinot/pull/12993#issuecomment-2073982624
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12993?dropdown=coverage=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache)
Report
xiangfu0 opened a new pull request, #12993:
URL: https://github.com/apache/pinot/pull/12993
1. Partition the data to containers first
2. For each partition, generate the window agg results. using merger
functions for now
--
This is an automated message from the Apache Git Service.
To
abhioncbr commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1577155394
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -128,13 +155,34 @@ public static PropertiesConfiguration fromFile(File
Jackie-Jiang commented on code in PR #12886:
URL: https://github.com/apache/pinot/pull/12886#discussion_r1577055850
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java:
##
@@ -608,172 +721,153 @@ protected SegmentDataManager
Jackie-Jiang commented on code in PR #12886:
URL: https://github.com/apache/pinot/pull/12886#discussion_r1577054823
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java:
##
@@ -608,172 +721,153 @@ protected SegmentDataManager
Jackie-Jiang commented on code in PR #12886:
URL: https://github.com/apache/pinot/pull/12886#discussion_r1577051333
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java:
##
@@ -608,172 +721,153 @@ protected SegmentDataManager
Jackie-Jiang commented on code in PR #12886:
URL: https://github.com/apache/pinot/pull/12886#discussion_r1577046142
##
pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java:
##
@@ -428,17 +570,25 @@ public ExecutorService
Jackie-Jiang merged PR #12392:
URL: https://github.com/apache/pinot/pull/12392
--
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:
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 0caeccfc13 Bump org.webjars:swagger-ui from 5.15.0 to 5.17.0 (#12989)
add 36c4b9a86f Add Prefix, Suffix and Ngram
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/org.webjars-swagger-ui-5.17.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 9ac0fa8507 Fix swagger resources path
The revisions that were on this branch
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 0caeccfc13 Bump org.webjars:swagger-ui from
Jackie-Jiang merged PR #12989:
URL: https://github.com/apache/pinot/pull/12989
--
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:
abhioncbr commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1576806764
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -43,28 +46,16 @@
*/
public class CommonsConfigurationUtils {
private
klsince commented on code in PR #12976:
URL: https://github.com/apache/pinot/pull/12976#discussion_r1576744981
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java:
##
@@ -1103,6 +1210,48 @@ private static
Jackie-Jiang commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1576704306
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -43,28 +46,16 @@
*/
public class CommonsConfigurationUtils {
Jackie-Jiang commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1576704306
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -43,28 +46,16 @@
*/
public class CommonsConfigurationUtils {
Jackie-Jiang commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1576701431
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -43,28 +46,16 @@
*/
public class CommonsConfigurationUtils {
Jackie-Jiang commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1576701431
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -43,28 +46,16 @@
*/
public class CommonsConfigurationUtils {
klsince commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1576653550
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -43,28 +46,16 @@
*/
public class CommonsConfigurationUtils {
private
klsince commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1576653550
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -43,28 +46,16 @@
*/
public class CommonsConfigurationUtils {
private
klsince commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1576653550
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -43,28 +46,16 @@
*/
public class CommonsConfigurationUtils {
private
klsince commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1576653550
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -43,28 +46,16 @@
*/
public class CommonsConfigurationUtils {
private
klsince commented on code in PR #12440:
URL: https://github.com/apache/pinot/pull/12440#discussion_r1576650089
##
pinot-spi/src/main/java/org/apache/pinot/spi/env/CommonsConfigurationUtils.java:
##
@@ -43,28 +46,16 @@
*/
public class CommonsConfigurationUtils {
private
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/org.webjars-swagger-ui-5.17.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
from c2c795c58a Bump org.webjars:swagger-ui from 5.15.0 to 5.17.0
add
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/aws.sdk.version-2.25.36
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 9e21da3528 Bump aws.sdk.version from 2.25.35 to 2.25.36
The revisions that were on
Jackie-Jiang merged PR #12990:
URL: https://github.com/apache/pinot/pull/12990
--
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:
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 40cf5a7ba8 Bump aws.sdk.version from 2.25.35 to
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/org.jline-jline-3.26.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
was cad0a671ea Remove wrong comment
The revisions that were on this branch are still
Jackie-Jiang merged PR #12991:
URL: https://github.com/apache/pinot/pull/12991
--
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:
Jackie-Jiang commented on code in PR #12991:
URL: https://github.com/apache/pinot/pull/12991#discussion_r1576650849
##
pom.xml:
##
@@ -244,7 +244,7 @@
3.9.0
2.0.3
Review Comment:
```suggestion
```
--
This is an automated message from the Apache Git
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a change to branch dependabot/maven/org.jline-jline-3.26.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 8bb3ae7d0e Bump org.jline:jline from 3.24.1 to 3.26.0
add cad0a671ea Remove
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 8e10320595 handle absent segments so that catchup checker doesn't get
stuck on them (#12883)
add dd4f0acabd Bump
yashmayya commented on code in PR #12992:
URL: https://github.com/apache/pinot/pull/12992#discussion_r1576609447
##
pinot-common/pom.xml:
##
@@ -62,6 +62,27 @@
protobuf-maven-plugin
+
+com.diffplug.spotless
+spotless-maven-plugin
+
deemoliu commented on code in PR #12392:
URL: https://github.com/apache/pinot/pull/12392#discussion_r1576631085
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -581,6 +584,111 @@ public static String[] split(String input, String
deemoliu commented on code in PR #12392:
URL: https://github.com/apache/pinot/pull/12392#discussion_r1576618558
##
pinot-common/src/main/java/org/apache/pinot/common/function/scalar/StringFunctions.java:
##
@@ -581,6 +584,111 @@ public static String[] split(String input, String
yashmayya commented on code in PR #12992:
URL: https://github.com/apache/pinot/pull/12992#discussion_r1576609447
##
pinot-common/pom.xml:
##
@@ -62,6 +62,27 @@
protobuf-maven-plugin
+
+com.diffplug.spotless
+spotless-maven-plugin
+
gortiz commented on code in PR #12992:
URL: https://github.com/apache/pinot/pull/12992#discussion_r1576604451
##
pinot-common/pom.xml:
##
@@ -62,6 +62,27 @@
protobuf-maven-plugin
+
+com.diffplug.spotless
+spotless-maven-plugin
+
Jackie-Jiang commented on code in PR #12992:
URL: https://github.com/apache/pinot/pull/12992#discussion_r1576601232
##
pinot-common/pom.xml:
##
@@ -62,6 +62,27 @@
protobuf-maven-plugin
+
+com.diffplug.spotless
+spotless-maven-plugin
+
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a commit to branch new-site-dev
in repository https://gitbox.apache.org/repos/asf/pinot-site.git
The following commit(s) were added to refs/heads/new-site-dev by this push:
new 087db244 WB-284 - Add more
xiangfu0 merged PR #121:
URL: https://github.com/apache/pinot-site/pull/121
--
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:
codecov-commenter commented on PR #12992:
URL: https://github.com/apache/pinot/pull/12992#issuecomment-2072735999
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12992?dropdown=coverage=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache)
Report
yashmayya opened a new pull request, #12992:
URL: https://github.com/apache/pinot/pull/12992
- The Spotless plugin was disabled for Java 21 in
https://github.com/apache/pinot/pull/11670 because Spotless didn't support Java
21 at the time.
- However,
gio-startree opened a new pull request, #121:
URL: https://github.com/apache/pinot-site/pull/121
This pull request adds 25 new company logos to the Powered By page. The
logos are in SVG format and have been included in the appropriate files.
--
This is an automated message from the
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1576220503
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/blocks/TransferableBlock.java:
##
@@ -19,44 +19,50 @@
package
mghildiy commented on issue #12645:
URL: https://github.com/apache/pinot/issues/12645#issuecomment-2072241205
Maybe I can give it a try if noone else is working on it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1576213860
##
pinot-query-runtime/src/main/java/org/apache/pinot/query/mailbox/ReceivingMailbox.java:
##
@@ -72,11 +81,53 @@ public String getId() {
return _id;
}
+ /**
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1576205973
##
pinot-common/src/main/java/org/apache/pinot/common/datablock/V1MetadataBlock.java:
##
@@ -0,0 +1,165 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF)
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1576200948
##
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/MultiStageBrokerRequestHandler.java:
##
@@ -276,6 +264,161 @@ protected BrokerResponse
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1576190502
##
pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/MultiStageBrokerRequestHandler.java:
##
@@ -276,6 +264,161 @@ protected BrokerResponse
gortiz commented on code in PR #12704:
URL: https://github.com/apache/pinot/pull/12704#discussion_r1576184899
##
pinot-common/src/main/java/org/apache/pinot/common/response/broker/BrokerResponseNative.java:
##
@@ -51,6 +59,7 @@
"segmentStatistics", "traceInfo",
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch dependabot/maven/org.jline-jline-3.25.1
in repository https://gitbox.apache.org/repos/asf/pinot.git
was ebfc61722a Bump org.jline:jline from 3.24.1 to 3.25.1
The revisions that were on
dependabot[bot] commented on PR #12875:
URL: https://github.com/apache/pinot/pull/12875#issuecomment-2072071073
Superseded by #12991.
--
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
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch dependabot/maven/aws.sdk.version-2.25.36
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 9e21da3528 Bump aws.sdk.version from 2.25.35 to 2.25.36
No new revisions were
dependabot[bot] opened a new pull request, #12990:
URL: https://github.com/apache/pinot/pull/12990
Bumps `aws.sdk.version` from 2.25.35 to 2.25.36.
Updates `software.amazon.awssdk:bom` from 2.25.35 to 2.25.36
Updates `software.amazon.awssdk:apache-client` from 2.25.35 to 2.25.36
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch dependabot/maven/org.jline-jline-3.26.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 8bb3ae7d0e Bump org.jline:jline from 3.24.1 to 3.26.0
No new revisions were
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/org.webjars-swagger-ui-5.17.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
at c2c795c58a Bump org.webjars:swagger-ui from 5.15.0 to 5.17.0
No new
dependabot[bot] closed pull request #12875: Bump org.jline:jline from 3.24.1 to
3.25.1
URL: https://github.com/apache/pinot/pull/12875
--
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] opened a new pull request, #12991:
URL: https://github.com/apache/pinot/pull/12991
Bumps [org.jline:jline](https://github.com/jline/jline3) from 3.24.1 to
3.26.0.
Release notes
Sourced from https://github.com/jline/jline3/releases;>org.jline:jline's
releases.
dependabot[bot] opened a new pull request, #12989:
URL: https://github.com/apache/pinot/pull/12989
Bumps [org.webjars:swagger-ui](https://github.com/swagger-api/swagger-ui)
from 5.15.0 to 5.17.0.
Release notes
Sourced from
tibrewalpratik17 commented on issue #12987:
URL: https://github.com/apache/pinot/issues/12987#issuecomment-2071920309
> Provide partition id externally:
Option 1: Provide partition id as http headers during segment upload
Option 2: Provide partition id as part of uploaded segment
hpvd commented on issue #10919:
URL: https://github.com/apache/pinot/issues/10919#issuecomment-2071881575
Release video: Apache Pinot 1.1 | Overview of Latest Features and Updates
https://www.youtube.com/watch?v=wSwPtOajsGY
talks also about vector index support:
prajyotkcsu commented on issue #11503:
URL: https://github.com/apache/pinot/issues/11503#issuecomment-2071478815
good observation!
--
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
69 matches
Mail list logo