[pinot] branch manual-authorization-annotation updated (f67ac4522d -> a569240c35)

2022-08-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch manual-authorization-annotation in repository https://gitbox.apache.org/repos/asf/pinot.git discard f67ac4522d controller auth interface cleanup discard d38c84cd02 annotation fix discard 1373

[GitHub] [pinot] codecov-commenter commented on pull request #9257: Fix a UT that only shows up on host with more cores

2022-08-19 Thread GitBox
codecov-commenter commented on PR #9257: URL: https://github.com/apache/pinot/pull/9257#issuecomment-1221186810 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9257?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] sajjad-moradi commented on pull request #9249: Gracefully handle problem in fetching stream partition count for consumption rate limiter

2022-08-19 Thread GitBox
sajjad-moradi commented on PR #9249: URL: https://github.com/apache/pinot/pull/9249#issuecomment-1221184780 > We had discussed about a metadata cache. is that something that is coming in another PR (and this one is to get us through the immediate problem)? > > thanks Yes that w

[GitHub] [pinot] klsince commented on a diff in pull request #9242: refine query cancel resp msg

2022-08-19 Thread GitBox
klsince commented on code in PR #9242: URL: https://github.com/apache/pinot/pull/9242#discussion_r950618560 ## pinot-core/src/main/java/org/apache/pinot/core/operator/combine/BaseCombineOperator.java: ## @@ -118,6 +119,9 @@ public void runJob() { IntermediateResultsBlock me

[GitHub] [pinot] klsince commented on a diff in pull request #9242: refine query cancel resp msg

2022-08-19 Thread GitBox
klsince commented on code in PR #9242: URL: https://github.com/apache/pinot/pull/9242#discussion_r950618452 ## pinot-common/src/main/java/org/apache/pinot/common/exception/QueryException.java: ## @@ -53,6 +53,7 @@ public static void setMaxLinesOfStackTrace(int maxLinesOfStackTr

[pinot] branch manual-authorization-annotation updated (d38c84cd02 -> f67ac4522d)

2022-08-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch manual-authorization-annotation in repository https://gitbox.apache.org/repos/asf/pinot.git from d38c84cd02 annotation fix add f67ac4522d controller auth interface cleanup No new revis

[GitHub] [pinot] Jackie-Jiang commented on a diff in pull request #9242: refine query cancel resp msg

2022-08-19 Thread GitBox
Jackie-Jiang commented on code in PR #9242: URL: https://github.com/apache/pinot/pull/9242#discussion_r950611815 ## pinot-common/src/main/java/org/apache/pinot/common/exception/QueryException.java: ## @@ -53,6 +53,7 @@ public static void setMaxLinesOfStackTrace(int maxLinesOfSt

[GitHub] [pinot] klsince opened a new pull request, #9257: Fix a UT that only shows up on host with more cores

2022-08-19 Thread GitBox
klsince opened a new pull request, #9257: URL: https://github.com/apache/pinot/pull/9257 Fix a UT that only shows up on host with more cores. The instance running GH Action couldn't catch it. I believe it has less than 6 cores. -- This is an automated message from the Apache Git Servi

[pinot] branch master updated: Add more failure info in RealtimeNonReplicaGroupSegmentAssignmentTest.testRelocateCompletedSegments (#9254)

2022-08-19 Thread jackie
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 a5a83aa81b Add more failure info in RealtimeNonR

[GitHub] [pinot] Jackie-Jiang merged pull request #9254: Add more failure info in RealtimeNonReplicaGroupSegmentAssignmentTest.testRelocateCompletedSegments

2022-08-19 Thread GitBox
Jackie-Jiang merged PR #9254: URL: https://github.com/apache/pinot/pull/9254 -- 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

[GitHub] [pinot] navina commented on a diff in pull request #9244: add a freshness based consumption status checker

2022-08-19 Thread GitBox
navina commented on code in PR #9244: URL: https://github.com/apache/pinot/pull/9244#discussion_r950603573 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/LLRealtimeSegmentDataManager.java: ## @@ -1440,6 +1440,17 @@ private void fetchLatestStreamOffset()

[pinot] branch master updated: Fix flakiness and cleanup SegmentLineageCleanupTest (#9256)

2022-08-19 Thread xiangfu
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 7af3edbd52 Fix flakiness and cleanup SegmentLine

[GitHub] [pinot] xiangfu0 merged pull request #9256: Fix flakiness and cleanup SegmentLineageCleanupTest

2022-08-19 Thread GitBox
xiangfu0 merged PR #9256: URL: https://github.com/apache/pinot/pull/9256 -- 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

[GitHub] [pinot] codecov-commenter commented on pull request #9252: add @ManualAuthorization annotation for non-standard endpoints

2022-08-19 Thread GitBox
codecov-commenter commented on PR #9252: URL: https://github.com/apache/pinot/pull/9252#issuecomment-1221164092 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9252?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] codecov-commenter commented on pull request #9256: Fix flakiness and cleanup SegmentLineageCleanupTest

2022-08-19 Thread GitBox
codecov-commenter commented on PR #9256: URL: https://github.com/apache/pinot/pull/9256#issuecomment-1221153569 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9256?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[pinot] branch manual-authorization-annotation updated (13736c79f9 -> d38c84cd02)

2022-08-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch manual-authorization-annotation in repository https://gitbox.apache.org/repos/asf/pinot.git from 13736c79f9 linter fix add d38c84cd02 annotation fix No new revisions were added by this

[pinot] branch master updated: Remove currentDataTableVersion from helm value (#9255)

2022-08-19 Thread xiangfu
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 0a272b2956 Remove currentDataTableVersion from h

[GitHub] [pinot] xiangfu0 merged pull request #9255: Remove currentDataTableVersion from helm value

2022-08-19 Thread GitBox
xiangfu0 merged PR #9255: URL: https://github.com/apache/pinot/pull/9255 -- 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

[GitHub] [pinot] codecov-commenter commented on pull request #9254: Add more failure info in RealtimeNonReplicaGroupSegmentAssignmentTest.testRelocateCompletedSegments

2022-08-19 Thread GitBox
codecov-commenter commented on PR #9254: URL: https://github.com/apache/pinot/pull/9254#issuecomment-1221129016 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9254?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] xiangfu0 opened a new pull request, #9255: Remove currentDataTableVersion from helm value

2022-08-19 Thread GitBox
xiangfu0 opened a new pull request, #9255: URL: https://github.com/apache/pinot/pull/9255 This is no longer needed as presto/trino side has upgraded the DataTable version -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[pinot] branch manual-authorization-annotation updated (5ff57efb8c -> 13736c79f9)

2022-08-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch manual-authorization-annotation in repository https://gitbox.apache.org/repos/asf/pinot.git from 5ff57efb8c linter fix add 13736c79f9 linter fix No new revisions were added by this upd

[GitHub] [pinot] vvivekiyer commented on pull request #9247: Fix bug with logging request headers

2022-08-19 Thread GitBox
vvivekiyer commented on PR #9247: URL: https://github.com/apache/pinot/pull/9247#issuecomment-1221118277 LGTM. -- 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 unsubscri

[pinot] branch manual-authorization-annotation updated (ee391e918b -> 5ff57efb8c)

2022-08-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch manual-authorization-annotation in repository https://gitbox.apache.org/repos/asf/pinot.git from ee391e918b add @ManualAuthorization annotation for non-standard endpoints add 5ff57efb8

[pinot] branch master updated: Refactor upsert quickstarts (#9246)

2022-08-19 Thread xiangfu
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 e6c2f5bd3b Refactor upsert quickstarts (#9246) e

[GitHub] [pinot] xiangfu0 merged pull request #9246: Refactor upsert quickstarts

2022-08-19 Thread GitBox
xiangfu0 merged PR #9246: URL: https://github.com/apache/pinot/pull/9246 -- 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

[GitHub] [pinot] xiangfu0 commented on a diff in pull request #9246: Refactor upsert quickstarts

2022-08-19 Thread GitBox
xiangfu0 commented on code in PR #9246: URL: https://github.com/apache/pinot/pull/9246#discussion_r950573035 ## pinot-tools/src/main/java/org/apache/pinot/tools/PartialUpsertQuickStart.java: ## @@ -123,7 +67,10 @@ public void execute() printStatus(Color.CYAN, "Query : " + q

[GitHub] [pinot] Jackie-Jiang opened a new pull request, #9254: Add more failure info in RealtimeNonReplicaGroupSegmentAssignmentTest.testRelocateCompletedSegments

2022-08-19 Thread GitBox
Jackie-Jiang opened a new pull request, #9254: URL: https://github.com/apache/pinot/pull/9254 For #9253 -- 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-m

[GitHub] [pinot] Jackie-Jiang opened a new issue, #9253: RealtimeNonReplicaGroupSegmentAssignmentTest is flaky

2022-08-19 Thread GitBox
Jackie-Jiang opened a new issue, #9253: URL: https://github.com/apache/pinot/issues/9253 ``` Error: RealtimeNonReplicaGroupSegmentAssignmentTest.testRelocateCompletedSegments:231 expected [true] but found [false] ``` It tests the behavior of Helix auto rebalance strategy (`

[GitHub] [pinot] codecov-commenter commented on pull request #9251: use default authorization behavior for zk user config

2022-08-19 Thread GitBox
codecov-commenter commented on PR #9251: URL: https://github.com/apache/pinot/pull/9251#issuecomment-1221089837 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9251?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] PrachiKhobragade commented on a diff in pull request #9249: Gracefully handle problem in fetching stream partition count for consumption rate limiter

2022-08-19 Thread GitBox
PrachiKhobragade commented on code in PR #9249: URL: https://github.com/apache/pinot/pull/9249#discussion_r950550273 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeConsumptionRateManager.java: ## @@ -131,4 +144,22 @@ public void throttle(int numM

[GitHub] [pinot] Jackie-Jiang commented on a diff in pull request #9247: Fix bug with logging request headers

2022-08-19 Thread GitBox
Jackie-Jiang commented on code in PR #9247: URL: https://github.com/apache/pinot/pull/9247#discussion_r950548988 ## pinot-broker/src/main/java/org/apache/pinot/broker/api/HttpRequesterIdentity.java: ## @@ -43,4 +45,31 @@ public String getEndpointUrl() { public void setEndpoin

[GitHub] [pinot] apucher opened a new pull request, #9252: add @ManualAuthorization annotation for non-standard endpoints

2022-08-19 Thread GitBox
apucher opened a new pull request, #9252: URL: https://github.com/apache/pinot/pull/9252 Several endpoints in pinot aren't served well by the AuthFilter's heuristics for extracting table names. In particular, this includes `/tableConfigs` and `/schema` endpoints that accept opaque payloads

[GitHub] [pinot] apucher opened a new pull request, #9251: use default authorization behavior for zk user config

2022-08-19 Thread GitBox
apucher opened a new pull request, #9251: URL: https://github.com/apache/pinot/pull/9251 Minor refactor of `PinotAccessControlUserRestletResource` to rely on the AuthFilter's default behavior rather than performing authorization manually. CC: original author @INNOCENT-BOY, any subtle

[pinot] 01/01: add @ManualAuthorization annotation for non-standard endpoints

2022-08-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch manual-authorization-annotation in repository https://gitbox.apache.org/repos/asf/pinot.git commit ee391e918b188e6e7e6dade5f11157b4c04cf1d4 Author: Alexander Pucher AuthorDate: Fri Aug 19 13:21:

[pinot] branch manual-authorization-annotation created (now ee391e918b)

2022-08-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch manual-authorization-annotation in repository https://gitbox.apache.org/repos/asf/pinot.git at ee391e918b add @ManualAuthorization annotation for non-standard endpoints This branch inclu

[pinot] 01/01: use default authorization behavior for zk user config

2022-08-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a commit to branch use-default-authorization-for-zk-user-config in repository https://gitbox.apache.org/repos/asf/pinot.git commit 76184bbfe29b845de0975451af423c0319d07126 Author: Alexander Pucher AuthorDate: Fri

[pinot] branch use-default-authorization-for-zk-user-config created (now 76184bbfe2)

2022-08-19 Thread apucher
This is an automated email from the ASF dual-hosted git repository. apucher pushed a change to branch use-default-authorization-for-zk-user-config in repository https://gitbox.apache.org/repos/asf/pinot.git at 76184bbfe2 use default authorization behavior for zk user config This branch in

[pinot] branch master updated: Fix NPE in gap-fill when order-by is not specified (#9250)

2022-08-19 Thread siddteotia
This is an automated email from the ASF dual-hosted git repository. siddteotia 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 6e6e0a4e86 Fix NPE in gap-fill when order-by

[GitHub] [pinot] siddharthteotia merged pull request #9250: Fix NPE in gap-fill when order-by is not specified

2022-08-19 Thread GitBox
siddharthteotia merged PR #9250: URL: https://github.com/apache/pinot/pull/9250 -- 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...@pino

[GitHub] [pinot] snleee commented on a diff in pull request #9249: Gracefully handle problem in fetching stream partition count for consumption rate limiter

2022-08-19 Thread GitBox
snleee commented on code in PR #9249: URL: https://github.com/apache/pinot/pull/9249#discussion_r950515885 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeConsumptionRateManager.java: ## @@ -85,17 +88,27 @@ public ConsumptionRateLimiter createRat

[GitHub] [pinot] sajjad-moradi commented on a diff in pull request #9249: Gracefully handle problem in fetching stream partition count for consumption rate limiter

2022-08-19 Thread GitBox
sajjad-moradi commented on code in PR #9249: URL: https://github.com/apache/pinot/pull/9249#discussion_r950500727 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeConsumptionRateManager.java: ## @@ -131,4 +144,22 @@ public void throttle(int numMsgs

[GitHub] [pinot] codecov-commenter commented on pull request #9250: Fix NPE in gap-fill when order-by is not specified

2022-08-19 Thread GitBox
codecov-commenter commented on PR #9250: URL: https://github.com/apache/pinot/pull/9250#issuecomment-1221018092 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9250?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] sajjad-moradi commented on a diff in pull request #9249: Gracefully handle problem in fetching stream partition count for consumption rate limiter

2022-08-19 Thread GitBox
sajjad-moradi commented on code in PR #9249: URL: https://github.com/apache/pinot/pull/9249#discussion_r950491202 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeConsumptionRateManager.java: ## @@ -85,17 +88,27 @@ public ConsumptionRateLimiter cr

[GitHub] [pinot] tanmesh commented on a diff in pull request #9184: Add support for EXTRACT syntax and converts it to appropriate Pinot expression

2022-08-19 Thread GitBox
tanmesh commented on code in PR #9184: URL: https://github.com/apache/pinot/pull/9184#discussion_r950488823 ## pinot-core/src/test/java/org/apache/pinot/core/operator/transform/function/ExtractTransformFunctionTest.java: ## @@ -0,0 +1,60 @@ +package org.apache.pinot.core.operato

[GitHub] [pinot] snleee commented on a diff in pull request #9249: Gracefully handle problem in fetching stream partition count for consumption rate limiter

2022-08-19 Thread GitBox
snleee commented on code in PR #9249: URL: https://github.com/apache/pinot/pull/9249#discussion_r950487153 ## pinot-core/src/main/java/org/apache/pinot/core/data/manager/realtime/RealtimeConsumptionRateManager.java: ## @@ -131,4 +144,22 @@ public void throttle(int numMsgs) {

[GitHub] [pinot] tanmesh commented on pull request #9184: Add support for EXTRACT syntax and converts it to appropriate Pinot expression

2022-08-19 Thread GitBox
tanmesh commented on PR #9184: URL: https://github.com/apache/pinot/pull/9184#issuecomment-1221006954 PTAL @Jackie-Jiang when you get a chance. Thanks! -- 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 g

[GitHub] [pinot] navina commented on pull request #9244: add a freshness based consumption status checker

2022-08-19 Thread GitBox
navina commented on PR #9244: URL: https://github.com/apache/pinot/pull/9244#issuecomment-1220985810 > I agree that this would likely be better as a table level check and maybe even happen continuously through the lifecycle of the server so the broker is aware what servers are fresh enough

[GitHub] [pinot] Jackie-Jiang closed issue #8819: Updating schema with new MV column causes index failures and stops consumption

2022-08-19 Thread GitBox
Jackie-Jiang closed issue #8819: Updating schema with new MV column causes index failures and stops consumption URL: https://github.com/apache/pinot/issues/8819 -- 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

[GitHub] [pinot] tanmesh commented on a diff in pull request #9184: [WIP] Add support for EXTRACT syntax and converts it to appropriate Pinot expression

2022-08-19 Thread GitBox
tanmesh commented on code in PR #9184: URL: https://github.com/apache/pinot/pull/9184#discussion_r950434935 ## pinot-common/src/main/java/org/apache/pinot/common/function/TransformFunctionType.java: ## @@ -96,6 +96,8 @@ public enum TransformFunctionType { LOOKUP("lookUp"),

[GitHub] [pinot] tanmesh commented on a diff in pull request #9184: [WIP] Add support for EXTRACT syntax and converts it to appropriate Pinot expression

2022-08-19 Thread GitBox
tanmesh commented on code in PR #9184: URL: https://github.com/apache/pinot/pull/9184#discussion_r950434236 ## pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java: ## @@ -752,6 +754,8 @@ private static Expression compileFunctionExpression(SqlBasicCall

[GitHub] [pinot] tanmesh commented on a diff in pull request #9184: [WIP] Add support for EXTRACT syntax and converts it to appropriate Pinot expression

2022-08-19 Thread GitBox
tanmesh commented on code in PR #9184: URL: https://github.com/apache/pinot/pull/9184#discussion_r950433406 ## pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/PinotClientTransport.java: ## @@ -42,4 +42,4 @@ Future executeQueryAsync(String brokerAddress, Re

[GitHub] [pinot] Jackie-Jiang commented on a diff in pull request #9244: add a freshness based consumption status checker

2022-08-19 Thread GitBox
Jackie-Jiang commented on code in PR #9244: URL: https://github.com/apache/pinot/pull/9244#discussion_r950412090 ## pinot-spi/src/main/java/org/apache/pinot/spi/utils/CommonConstants.java: ## @@ -385,6 +385,13 @@ public static class Server { "pinot.server.starter.enabl

[GitHub] [pinot] codecov-commenter commented on pull request #9249: Gracefully handle problem in fetching stream partition count for consumption rate limiter

2022-08-19 Thread GitBox
codecov-commenter commented on PR #9249: URL: https://github.com/apache/pinot/pull/9249#issuecomment-1220908246 # [Codecov](https://codecov.io/gh/apache/pinot/pull/9249?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Softwar

[GitHub] [pinot] sajjad-moradi opened a new pull request, #9249: Gracefully handle problem in fetching stream partition count for consumption rate limiter

2022-08-19 Thread GitBox
sajjad-moradi opened a new pull request, #9249: URL: https://github.com/apache/pinot/pull/9249 Setting up consumption rate limit requires knowing the stream partition count so that each partition rate limit can be calculated based on the topic consumption rate limit config. Connecting to st

[GitHub] [pinot] yupeng9 commented on a diff in pull request #9246: Refactor upsert quickstarts

2022-08-19 Thread GitBox
yupeng9 commented on code in PR #9246: URL: https://github.com/apache/pinot/pull/9246#discussion_r950345665 ## pinot-tools/src/main/java/org/apache/pinot/tools/PartialUpsertQuickStart.java: ## @@ -123,7 +67,10 @@ public void execute() printStatus(Color.CYAN, "Query : " + q2

[GitHub] [pinot] jadami10 commented on pull request #9244: add a freshness based consumption status checker

2022-08-19 Thread GitBox
jadami10 commented on PR #9244: URL: https://github.com/apache/pinot/pull/9244#issuecomment-1220771563 Hey @navina, thank you for looking! I agree that this would likely be better as a table level check and maybe even happen continuously through the lifecycle of the server so the brok

[GitHub] [pinot] walterddr commented on a diff in pull request #9245: [Feature] List files from inputDir with recursive option based on inputFilePatterns in IngestionJobLauncher

2022-08-19 Thread GitBox
walterddr commented on code in PR #9245: URL: https://github.com/apache/pinot/pull/9245#discussion_r950261062 ## pinot-plugins/pinot-batch-ingestion/pinot-batch-ingestion-standalone/src/main/java/org/apache/pinot/plugin/ingestion/batch/standalone/SegmentGenerationJobRunner.java: ###

[pinot] branch dependabot/maven/org.apache.maven.plugins-maven-enforcer-plugin-3.1.0 created (now a7a52f69ac)

2022-08-19 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/org.apache.maven.plugins-maven-enforcer-plugin-3.1.0 in repository https://gitbox.apache.org/repos/asf/pinot.git at a7a52f69ac Bump maven-enforcer-plugin from 3.0.0-M3

[GitHub] [pinot] dependabot[bot] opened a new pull request, #9248: Bump maven-enforcer-plugin from 3.0.0-M3 to 3.1.0

2022-08-19 Thread GitBox
dependabot[bot] opened a new pull request, #9248: URL: https://github.com/apache/pinot/pull/9248 Bumps [maven-enforcer-plugin](https://github.com/apache/maven-enforcer) from 3.0.0-M3 to 3.1.0. Release notes Sourced from https://github.com/apache/maven-enforcer/releases";>maven-enfo

[GitHub] [pinot] gortiz commented on a diff in pull request #8979: optimize `order by sorted ASC, unsorted` and `order by DESC` cases

2022-08-19 Thread GitBox
gortiz commented on code in PR #8979: URL: https://github.com/apache/pinot/pull/8979#discussion_r950061067 ## pinot-core/src/main/java/org/apache/pinot/core/operator/blocks/IntermediateResultsBlock.java: ## @@ -80,9 +81,8 @@ public IntermediateResultsBlock() { /** * Const

[GitHub] [pinot] xiangfu0 commented on issue #9147: Latest docker image doesn't include arm64

2022-08-19 Thread GitBox
xiangfu0 commented on issue #9147: URL: https://github.com/apache/pinot/issues/9147#issuecomment-1220478638 I've added a nightly build image for arm64: https://github.com/apachepinot/pinot-fork/actions/workflows/nightly-build-pinot-arm64-platform-docker-image.yml It takes about 5 h