suddendust opened a new pull request, #12073:
URL: https://github.com/apache/pinot/pull/12073
Recently, we detected that the catch-all label was incorrect - It failed to
capture a metric for which a specific regex had not been added. It also does
not capture metrics that contain the kafka t
Jackie-Jiang commented on code in PR #12037:
URL: https://github.com/apache/pinot/pull/12037#discussion_r1410118166
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/ConcurrentMapPartitionUpsertMetadataManager.java:
##
@@ -323,11 +347,13 @@ static class
Jackie-Jiang commented on code in PR #12037:
URL: https://github.com/apache/pinot/pull/12037#discussion_r1410119462
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/ConcurrentMapPartitionUpsertMetadataManager.java:
##
@@ -323,11 +347,13 @@ static class
Jackie-Jiang commented on code in PR #12037:
URL: https://github.com/apache/pinot/pull/12037#discussion_r1410118166
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/ConcurrentMapPartitionUpsertMetadataManager.java:
##
@@ -323,11 +347,13 @@ static class
walterddr commented on code in PR #12049:
URL: https://github.com/apache/pinot/pull/12049#discussion_r1410057271
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/partition/Murmur3PartitionFunction.java:
##
@@ -0,0 +1,69 @@
+/**
+ * Licensed to the Apache Software
walterddr commented on code in PR #12049:
URL: https://github.com/apache/pinot/pull/12049#discussion_r1410057271
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/partition/Murmur3PartitionFunction.java:
##
@@ -0,0 +1,69 @@
+/**
+ * Licensed to the Apache Software
walterddr commented on code in PR #12049:
URL: https://github.com/apache/pinot/pull/12049#discussion_r1410055786
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/partition/Murmur3PartitionFunction.java:
##
@@ -0,0 +1,69 @@
+/**
+ * Licensed to the Apache Software
walterddr commented on code in PR #12049:
URL: https://github.com/apache/pinot/pull/12049#discussion_r1410054731
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/partition/Murmur3PartitionFunction.java:
##
@@ -0,0 +1,69 @@
+/**
+ * Licensed to the Apache Software
walterddr commented on code in PR #12049:
URL: https://github.com/apache/pinot/pull/12049#discussion_r1410054731
##
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/partition/Murmur3PartitionFunction.java:
##
@@ -0,0 +1,69 @@
+/**
+ * Licensed to the Apache Software
walterddr commented on code in PR #12058:
URL: https://github.com/apache/pinot/pull/12058#discussion_r1410047421
##
pinot-query-planner/src/main/java/org/apache/pinot/query/QueryEnvironment.java:
##
@@ -122,7 +122,8 @@ public QueryEnvironment(TypeFactory typeFactory,
CalciteSch
Jackie-Jiang commented on code in PR #11953:
URL: https://github.com/apache/pinot/pull/11953#discussion_r1410042951
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/assignment/instance/InstanceTagPoolSelector.java:
##
@@ -109,11 +121,46 @@ public Map>
se
codecov-commenter commented on PR #12072:
URL: https://github.com/apache/pinot/pull/12072#issuecomment-1832925957
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12072?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
Jackie-Jiang opened a new pull request, #12072:
URL: https://github.com/apache/pinot/pull/12072
Address #11895
Introduce a new boolean config `lowDiskMode` (by default false) to rebalance
config. When enabled, it will offload segments first, then add segments after
offload is done.
klsince commented on code in PR #11847:
URL: https://github.com/apache/pinot/pull/11847#discussion_r1367558654
##
pinot-broker/src/main/java/org/apache/pinot/broker/routing/instanceselector/FallbackReplicaGroupInstanceSelector.java:
##
@@ -0,0 +1,428 @@
+/**
+ * Licensed to the
codecov-commenter commented on PR #12071:
URL: https://github.com/apache/pinot/pull/12071#issuecomment-1832833341
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12071?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
wirybeaver opened a new pull request, #12071:
URL: https://github.com/apache/pinot/pull/12071
Leverage the modular arithmetic to avoid code duplication.
--
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
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 b9ed378355 [multistage][refactor] clean up planner
walterddr merged PR #12070:
URL: https://github.com/apache/pinot/pull/12070
--
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
codecov-commenter commented on PR #12070:
URL: https://github.com/apache/pinot/pull/12070#issuecomment-1832613717
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12070?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
walterddr opened a new pull request, #12070:
URL: https://github.com/apache/pinot/pull/12070
- colocated visitor is no longer needed, we derived using partition now
- plan fragment metadata is not needed, we use dispatchablePlanFragment &
dispatchablePlanMetadata
- move several physic
klsince commented on code in PR #12039:
URL: https://github.com/apache/pinot/pull/12039#discussion_r1409728866
##
pinot-common/src/main/java/org/apache/pinot/common/utils/ServiceStartableUtils.java:
##
@@ -36,56 +40,91 @@ private ServiceStartableUtils() {
private static fin
JeffBolle commented on issue #12067:
URL: https://github.com/apache/pinot/issues/12067#issuecomment-1832385241
Documentation PR here: https://github.com/pinot-contrib/pinot-docs/pull/263
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
goodseeyou commented on PR #12069:
URL: https://github.com/apache/pinot/pull/12069#issuecomment-1832180673
Any instruction to add labels? If it's necessary. Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
goodseeyou opened a new pull request, #12069:
URL: https://github.com/apache/pinot/pull/12069
This PR is to correct default configure path for log4j in helm chart.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
codecov-commenter commented on PR #12068:
URL: https://github.com/apache/pinot/pull/12068#issuecomment-1832145059
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12068?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
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 c57117a750 [multistage] don't prune project after
walterddr closed issue #12013: [multistage][bug] multi semi-join hint not
applied in weird conditions
URL: https://github.com/apache/pinot/issues/12013
--
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 t
walterddr merged PR #12058:
URL: https://github.com/apache/pinot/pull/12058
--
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 merged PR #12052:
URL: https://github.com/apache/pinot/pull/12052
--
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 37270f749c [multistage][cleanup] remove some unuse
JeffBolle commented on issue #12067:
URL: https://github.com/apache/pinot/issues/12067#issuecomment-1832027306
PR here: https://github.com/apache/pinot/pull/12068
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
JeffBolle opened a new pull request, #12068:
URL: https://github.com/apache/pinot/pull/12068
Adds support for StreamNative OAuth2 authentication, allowing Pinot to
connect to StreamNative hosted Pulsar cluster.
fixes #12067
--
This is an automated message from the Apache Git Servic
codecov-commenter commented on PR #12066:
URL: https://github.com/apache/pinot/pull/12066#issuecomment-1831817325
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12066?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 #12065:
URL: https://github.com/apache/pinot/pull/12065#issuecomment-1831813130
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/12065?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
R
This is an automated email from the ASF dual-hosted git repository.
github-bot pushed a change to branch
dependabot/maven/com.adobe.testing-s3mock-testcontainers-3.2.0
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 76f7e19f0b Bump com.adobe.testing:s3mock-testcontainers fr
dependabot[bot] opened a new pull request, #12066:
URL: https://github.com/apache/pinot/pull/12066
Bumps net.alchim31.maven:scala-maven-plugin from 3.2.2 to 4.8.1.
[ of the filter expression must be the leaf level
of the JSON object, for example, "$.addresses[*]"='main st' won't work.
In some sce
davecromberge commented on code in PR #12042:
URL: https://github.com/apache/pinot/pull/12042#discussion_r1408415739
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/CustomSerDeUtils.java:
##
@@ -243,9 +243,14 @@ public TDigest deserialize(ByteBuffer byt
davecromberge commented on code in PR #12042:
URL: https://github.com/apache/pinot/pull/12042#discussion_r1408414659
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/customobject/ThetaSketchAccumulator.java:
##
@@ -0,0 +1,133 @@
+/**
+ * Licensed to the Apache
41 matches
Mail list logo
