suyashpatel98 opened a new pull request, #13320:
URL: https://github.com/apache/pinot/pull/13320
Fix URI construction so that AddSchema command line tool works when override
flag is set to true
Verified that when the override flag is set to true the schema is
overridden successfull
This is an automated email from the ASF dual-hosted git repository.
vvivekiyer pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from f4a4bf389a Custom configuration property reader for segment metadata
files (#12440)
add 6290bc001a Interning
vvivekiyer merged PR #12342:
URL: https://github.com/apache/pinot/pull/12342
--
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
abhioncbr commented on code in PR #13297:
URL: https://github.com/apache/pinot/pull/13297#discussion_r1628677484
##
pom.xml:
##
@@ -224,12 +224,12 @@
26.40.0
-1.23.0
-2.10.1
-33.1.0-jre
-1.44.1
-3.25.2
-1.61.1
+3.25.2
+1.61.1
Revi
Jackie-Jiang commented on issue #11448:
URL: https://github.com/apache/pinot/issues/11448#issuecomment-2151192409
1. What is the typical freshness lag during segment commit? If we don't want
to read IS for every segment commit, we need to configure a timeout longer than
the common freshness
klsince commented on code in PR #13314:
URL: https://github.com/apache/pinot/pull/13314#discussion_r1628544264
##
pinot-controller/src/main/java/org/apache/pinot/controller/util/CompletionServiceHelper.java:
##
@@ -129,13 +129,21 @@ private CompletionServiceResponse collectRespo
suyashpatel98 commented on issue #13312:
URL: https://github.com/apache/pinot/issues/13312#issuecomment-2151061084
@Jackie-Jiang Please assign this to me. I will raise the PR soon
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHu
jadami10 commented on issue #11448:
URL: https://github.com/apache/pinot/issues/11448#issuecomment-2151049470
I wish there was a smaller ZK node we could look at. But given that we (and
likely others) have use cases that want to meet SLC thresholds < 30s, then
that's roughly how quickly we'
Jackie-Jiang commented on code in PR #13297:
URL: https://github.com/apache/pinot/pull/13297#discussion_r1628489765
##
pom.xml:
##
@@ -1395,7 +1389,7 @@
net.openhft
posix
-2.26ea0
+2.25ea0
Review Comment:
Is this change intended?
Jackie-Jiang commented on issue #11448:
URL: https://github.com/apache/pinot/issues/11448#issuecomment-2151037009
Server doesn't read IS on its own. Controller periodically scan all IS/EV
and send messages to servers.
In order to detect that the next segment doesn't come to this server, s
jadami10 commented on issue #11448:
URL: https://github.com/apache/pinot/issues/11448#issuecomment-2151032441
@Jackie-Jiang, I wasn't able to track this down, but do servers consistently
parse changes between idealstate/external view? Is there a way to do it when
that happens and have it al
Jackie-Jiang commented on issue #11448:
URL: https://github.com/apache/pinot/issues/11448#issuecomment-2151027072
The current solution should be able to eventually remove the lag, but could
take up to 15 minutes (10 minutes timeout + 5 minutes scheduling delay).
There are 2 potential enha
klsince commented on code in PR #13314:
URL: https://github.com/apache/pinot/pull/13314#discussion_r1628481078
##
pinot-controller/src/main/java/org/apache/pinot/controller/util/ServerSegmentMetadataReader.java:
##
@@ -235,8 +236,18 @@ public List
getValidDocIdsMetadataFromServ
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 f4a4bf389a Custom configuration property reader for
klsince merged PR #12440:
URL: https://github.com/apache/pinot/pull/12440
--
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.
jiaguo 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 3d07b90bd7 Fix Logging Location for CPU-Based Que
jasperjiaguo merged PR #13318:
URL: https://github.com/apache/pinot/pull/13318
--
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 #13318:
URL: https://github.com/apache/pinot/pull/13318#issuecomment-2150896993
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13318?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
lnbest0707-uber opened a new issue, #13319:
URL: https://github.com/apache/pinot/issues/13319
Pinot data ingestion from Kafka is following the 1 thread per Kafka
partition mechanism. The scaling up is relying on increasing number of Kafka
topic partitions. However, due to the nature of inge
PraveenVora opened a new pull request, #13318:
URL: https://github.com/apache/pinot/pull/13318
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`
klsince commented on code in PR #13107:
URL: https://github.com/apache/pinot/pull/13107#discussion_r1628157120
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/creator/name/UploadedRealtimeSegmentNameGenerator.java:
##
@@ -0,0 +1,62 @@
+/**
+ * Licensed to the Apa
Jackie-Jiang commented on PR #13274:
URL: https://github.com/apache/pinot/pull/13274#issuecomment-2150569516
@dependabot rebase
--
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
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/net.openhft-chronicle-core-2.26ea0
in repository https://gitbox.apache.org/repos/asf/pinot.git
discard 65963672c1 Bump net.openhft:chronicle-core from 2.25ea15 to 2.25.16
Jackie-Jiang closed pull request #13252: Disabling checkstyle plugin and
Spotless plugin for Java version > 21.0
URL: https://github.com/apache/pinot/pull/13252
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
Jackie-Jiang commented on code in PR #13035:
URL: https://github.com/apache/pinot/pull/13035#discussion_r1628117693
##
pinot-common/src/main/java/org/apache/pinot/common/metrics/ServerTimer.java:
##
@@ -54,7 +54,14 @@ public enum ServerTimer implements AbstractMetrics.Timer {
abhioncbr closed pull request #13297: Clean Google Dependencies
URL: https://github.com/apache/pinot/pull/13297
--
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,
aishwaryathondapu commented on issue #13312:
URL: https://github.com/apache/pinot/issues/13312#issuecomment-2150541038
yes.
--
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.
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.25.66
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 1e52d710af Bump software.amazon.awssdk:bom from 2.25.64 to 2.25.66
The r
Jackie-Jiang merged PR #13316:
URL: https://github.com/apache/pinot/pull/13316
--
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
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 065f325b53 Bump software.amazon.awssdk:bom from 2
shounakmk219 commented on PR #13290:
URL: https://github.com/apache/pinot/pull/13290#issuecomment-2150188180
## Controller metrics validations:
1.
https://github.com/apache/pinot/assets/25409127/cb1a6438-8797-4de0-99f1-cff12ec7bd51";>
2.
https://github.com/apache/pinot/ass
shounakmk219 commented on code in PR #13290:
URL: https://github.com/apache/pinot/pull/13290#discussion_r1627830416
##
docker/images/pinot/etc/jmx_prometheus_javaagent/configs/server.yml:
##
@@ -73,6 +66,14 @@ rules:
table: "$1$3"
tableType: "$4"
partition: "$5"
+
shounakmk219 commented on code in PR #13290:
URL: https://github.com/apache/pinot/pull/13290#discussion_r1627823043
##
docker/images/pinot/etc/jmx_prometheus_javaagent/configs/broker.yml:
##
@@ -112,6 +118,12 @@ rules:
- pattern:
"\"org\\.apache\\.pinot\\.common\\.metrics\"<>(
shounakmk219 commented on code in PR #13290:
URL: https://github.com/apache/pinot/pull/13290#discussion_r1627809416
##
docker/images/pinot/etc/jmx_prometheus_javaagent/configs/broker.yml:
##
@@ -59,6 +59,12 @@ rules:
labels:
database: "$2"
table: "$1$3"
+- pattern:
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.25.65
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 874a2ddb43 Bump software.amazon.awssdk:bom from 2.25.64 to 2.25.65
T
dependabot[bot] closed pull request #13311: Bump software.amazon.awssdk:bom
from 2.25.64 to 2.25.65
URL: https://github.com/apache/pinot/pull/13311
--
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/software.amazon.awssdk-bom-2.25.66
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 1e52d710af Bump software.amazon.awssdk:bom from 2.25.64 to 2.25.66
N
dependabot[bot] commented on PR #13311:
URL: https://github.com/apache/pinot/pull/13311#issuecomment-2149608954
Superseded by #13316.
--
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, #13316:
URL: https://github.com/apache/pinot/pull/13316
Bumps software.amazon.awssdk:bom from 2.25.64 to 2.25.66.
[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=softwa
rohityadav1993 commented on code in PR #13256:
URL: https://github.com/apache/pinot/pull/13256#discussion_r1627386271
##
pinot-spi/src/main/java/org/apache/pinot/spi/config/table/UpsertConfig.java:
##
@@ -94,6 +94,9 @@ public enum ConsistencyMode {
@JsonPropertyDescription("W
gortiz commented on PR #13313:
URL: https://github.com/apache/pinot/pull/13313#issuecomment-2149310502
I've created a NOOP registry in https://github.com/apache/pinot/pull/13032,
so this kind of issues shouldn't be problematic in the future.
--
This is an automated message from the Apache
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 bdef83b749 Add metrics to count joins and window
gortiz merged PR #13032:
URL: https://github.com/apache/pinot/pull/13032
--
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
codecov-commenter commented on PR #13035:
URL: https://github.com/apache/pinot/pull/13035#issuecomment-2149272592
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13035?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
44 matches
Mail list logo
