xiangfu0 merged PR #13669:
URL: https://github.com/apache/pinot/pull/13669
--
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 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 66c8dfef78 Support apply logical type recursivel
xiangfu0 closed issue #13670: AvroSchemaUtils doesn't apply logical type in the
nested structure
URL: https://github.com/apache/pinot/issues/13670
--
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
kirkrodrigues opened a new pull request, #13687:
URL: https://github.com/apache/pinot/pull/13687
This updates to the latest release of clp-ffi that has support for Linux and
macOS ARM platforms.
This partly addresses #10945.
# Validation performed
On an Apple Silicon Mac
This is an automated email from the ASF dual-hosted git repository.
xiangfu pushed a change to branch SegmentProcessorFrameworkImprovement
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 5ef41d81bd Remove flaky test: testUpdateAndRemoveGauge (#13684)
No new revisions were a
shounakmk219 commented on PR #13544:
URL: https://github.com/apache/pinot/pull/13544#issuecomment-2249434537
> Are there tests for setting/updating qps due to helix messages for table
qps ? Do you plan to tests those out as well ?
@vrajat I am not sure how to specifically test that, w
shounakmk219 commented on code in PR #13544:
URL: https://github.com/apache/pinot/pull/13544#discussion_r1690805600
##
pinot-broker/src/test/java/org/apache/pinot/broker/queryquota/HelixExternalViewBasedQueryQuotaManagerTest.java:
##
@@ -82,21 +93,13 @@ private HelixManager init
vrajat commented on code in PR #13544:
URL: https://github.com/apache/pinot/pull/13544#discussion_r1690780744
##
pinot-spi/src/main/java/org/apache/pinot/spi/config/table/QuotaConfig.java:
##
@@ -35,6 +35,7 @@ public class QuotaConfig extends BaseJsonConfig {
private static f
KKcorps commented on code in PR #13668:
URL: https://github.com/apache/pinot/pull/13668#discussion_r1690763064
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java:
##
@@ -1029,6 +1039,47 @@ void
updateInstanceSta
KKcorps commented on code in PR #13649:
URL: https://github.com/apache/pinot/pull/13649#discussion_r1690763727
##
pinot-plugins/pinot-stream-ingestion/pinot-kinesis/src/main/java/org/apache/pinot/plugin/stream/kinesis/KinesisConfig.java:
##
@@ -70,7 +70,12 @@ public class Kinesi
itschrispeck commented on PR #13541:
URL: https://github.com/apache/pinot/pull/13541#issuecomment-2249074665
> My bad, now I see the problem. Should we consider first looping over the
ideal state and create a map from serving partition to state, then only loop
over this map in the second lo
codecov-commenter commented on PR #13686:
URL: https://github.com/apache/pinot/pull/13686#issuecomment-2248945603
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13686?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.
jlli 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 5ef41d81bd Remove flaky test: testUpdateAndRemoveGa
codecov-commenter commented on PR #13684:
URL: https://github.com/apache/pinot/pull/13684#issuecomment-2248901059
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13684?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
jackjlli merged PR #13684:
URL: https://github.com/apache/pinot/pull/13684
--
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 opened a new pull request, #13686:
URL: https://github.com/apache/pinot/pull/13686
Fixes #13683
Adds a new broker / table level config: newSegmentExpirationTimeInSeconds
which will help in configuring the maximum amount of time broker will wait for
new consuming seg
tibrewalpratik17 opened a new issue, #13685:
URL: https://github.com/apache/pinot/issues/13685
The
[AbstractMetrics](https://github.com/apache/pinot/blob/master/pinot-common/src/main/java/org/apache/pinot/common/metrics/AbstractMetrics.java)
class has very less related UTs -
https://github
tibrewalpratik17 opened a new pull request, #13684:
URL: https://github.com/apache/pinot/pull/13684
Remove flaky test introduced in #13679. This test was not testing as well of
what we want to so removing for now.
We need more UTs for `AbstractMetrics` class and created an issue for
tibrewalpratik17 commented on issue #13683:
URL: https://github.com/apache/pinot/issues/13683#issuecomment-2248804579
Hey @rohityadav1993 yes looks like #13256 might be the reason. I see the new
consuming thread blocked for 5-6 minutes when the old consuming segment was
getting replaced by
rohityadav1993 commented on issue #13683:
URL: https://github.com/apache/pinot/issues/13683#issuecomment-2248641953
Thanks @tibrewalpratik17 for the analysis. This could be aggravated due to
https://github.com/apache/pinot/pull/13256.
Configuring NEW_SEGMENT_EXPIRATION_MILLIS can be a
itschrispeck commented on code in PR #13503:
URL: https://github.com/apache/pinot/pull/13503#discussion_r1690224690
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/realtime/impl/invertedindex/RealtimeLuceneIndexRefreshManager.java:
##
@@ -0,0 +1,268 @@
+/**
+
tibrewalpratik17 opened a new issue, #13683:
URL: https://github.com/apache/pinot/issues/13683
Recently, we experienced an incident in our production environment where
numerous query failures occurred due to the absence of a replica group to route
the queries to. We are using the `strictRep
chenboat commented on code in PR #13503:
URL: https://github.com/apache/pinot/pull/13503#discussion_r1690197552
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/realtime/impl/invertedindex/RealtimeLuceneIndexRefreshManager.java:
##
@@ -0,0 +1,268 @@
+/**
+ * L
This is an automated email from the ASF dual-hosted git repository.
jlli 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 00871f242f Refactor PinotSchemaRestletResource for
jackjlli merged PR #13680:
URL: https://github.com/apache/pinot/pull/13680
--
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.
mcvsubbu pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
from 870cf57635 Fix flaky gauge value issue (#13679)
add c603cd47f7 Fix query response time metric (#13661)
No new r
mcvsubbu merged PR #13661:
URL: https://github.com/apache/pinot/pull/13661
--
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
codecov-commenter commented on PR #13645:
URL: https://github.com/apache/pinot/pull/13645#issuecomment-2247874775
##
[Codecov](https://app.codecov.io/gh/apache/pinot/pull/13645?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u
KKcorps commented on code in PR #13649:
URL: https://github.com/apache/pinot/pull/13649#discussion_r1689646609
##
pinot-plugins/pinot-stream-ingestion/pinot-kinesis/src/main/java/org/apache/pinot/plugin/stream/kinesis/KinesisConfig.java:
##
@@ -70,7 +70,12 @@ public class Kinesi
KKcorps commented on code in PR #13649:
URL: https://github.com/apache/pinot/pull/13649#discussion_r1689643390
##
pinot-plugins/pinot-stream-ingestion/pinot-kinesis/src/main/java/org/apache/pinot/plugin/stream/kinesis/KinesisConsumer.java:
##
@@ -122,7 +131,7 @@ private void rat
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.26.22
in repository https://gitbox.apache.org/repos/asf/pinot.git
was 71126e369d Bump software.amazon.awssdk:bom from 2.26.21 to 2.26.22
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.26.23
in repository https://gitbox.apache.org/repos/asf/pinot.git
at 909a217047 Bump software.amazon.awssdk:bom from 2.26.21 to 2.26.23
N
dependabot[bot] opened a new pull request, #13682:
URL: https://github.com/apache/pinot/pull/13682
Bumps software.amazon.awssdk:bom from 2.26.21 to 2.26.23.
[