(pinot) branch master updated: Use Picocli to generate help (#14419)

2024-11-11 Thread gortiz
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 a458426a32 Use Picocli to generate help (#14419)

Re: [PR] Use Picocli to generate help [pinot]

2024-11-11 Thread via GitHub
gortiz merged PR #14419: URL: https://github.com/apache/pinot/pull/14419 -- 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

Re: [I] Caused by: org.apache.kafka.common.errors.TimeoutException: Timeout expired while fetching topic metadata [pinot]

2024-11-11 Thread via GitHub
Abdulsamirkhan commented on issue #14372: URL: https://github.com/apache/pinot/issues/14372#issuecomment-2469727092 @Jackie-Jiang @swaminathanmanish @KKcorps, Still i am not able to solve this any help much appreciated. -- This is an automated message from the Apache Git Service. To respo

Re: [PR] Add `DATE_TRUNC` Optimizer [pinot]

2024-11-11 Thread via GitHub
jadami10 commented on code in PR #14385: URL: https://github.com/apache/pinot/pull/14385#discussion_r1837438040 ## pinot-core/src/main/java/org/apache/pinot/core/query/optimizer/filter/TimePredicateFilterOptimizer.java: ## @@ -411,6 +440,95 @@ && isStringLiteral(dateTimeConvertO

Re: [PR] [timeseries] Fix Server Selection Bug + Enforce Timeout [pinot]

2024-11-11 Thread via GitHub
codecov-commenter commented on PR #14426: URL: https://github.com/apache/pinot/pull/14426#issuecomment-2469346819 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/14426?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

Re: [PR] [timeseries] Fix Server Selection Bug + Enforce Timeout [pinot]

2024-11-11 Thread via GitHub
ankitsultana commented on code in PR #14426: URL: https://github.com/apache/pinot/pull/14426#discussion_r1837315362 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/QueryRunner.java: ## @@ -415,4 +399,31 @@ private PlanNode substituteNode(PlanNode node, Map

[PR] [timeseries] Fix Server Selection Bug + Enforce Timeout [pinot]

2024-11-11 Thread via GitHub
ankitsultana opened a new pull request, #14426: URL: https://github.com/apache/pinot/pull/14426 ### Summary We found that at present time-series queries were always going to the same server. This is because requestId wasn't set in the RequestContext. This PR fixes that. Additi

(pinot) branch master updated: [timeseries] Make Time Buckets Half Open on the Left (#14413)

2024-11-11 Thread ankitsultana
This is an automated email from the ASF dual-hosted git repository. ankitsultana 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 c5fbe5b23d [timeseries] Make Time Buckets H

Re: [PR] [timeseries] Make Time Buckets Half Open on the Left [pinot]

2024-11-11 Thread via GitHub
ankitsultana merged PR #14413: URL: https://github.com/apache/pinot/pull/14413 -- 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

Re: [I] Pauseless Consumption [pinot]

2024-11-11 Thread via GitHub
mcvsubbu commented on issue #10147: URL: https://github.com/apache/pinot/issues/10147#issuecomment-2468880798 @9aman please refer to the comment https://github.com/apache/pinot/issues/10147#issuecomment-2197768243 in this issue. We should not be updating the idealstate before the segment h

Re: [I] Cleanup `String.format()` usage [pinot]

2024-11-11 Thread via GitHub
ashishjayamohan commented on issue #14404: URL: https://github.com/apache/pinot/issues/14404#issuecomment-2468811929 That makes sense to me as well, just figured I'd check in with you before making that change. I've refactored a lot of usages in my PR. Let me know if anything looks off! -

Re: [PR] [WIP] Add a constructor to pass custom transform pipelines to SegmentMapper [pinot]

2024-11-11 Thread via GitHub
codecov-commenter commented on PR #14425: URL: https://github.com/apache/pinot/pull/14425#issuecomment-2468214135 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/14425?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

Re: [PR] Cleanup UpsertCompaction validations during task-generation [pinot]

2024-11-11 Thread via GitHub
codecov-commenter commented on PR #14424: URL: https://github.com/apache/pinot/pull/14424#issuecomment-2468181124 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/14424?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

[PR] [WIP] Add a constructor to pass custom transform pipelines to SegmentMapper [pinot]

2024-11-11 Thread via GitHub
shounakmk219 opened a new pull request, #14425: URL: https://github.com/apache/pinot/pull/14425 (no comment) -- 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,

[PR] Cleanup UpsertCompaction validations during task-generation [pinot]

2024-11-11 Thread via GitHub
tibrewalpratik17 opened a new pull request, #14424: URL: https://github.com/apache/pinot/pull/14424 A small follow-up to #14340. Now that we have moved validations to table-config creation / modification, removing the redundancy during task-generation process. cc @swaminathanmanish

Re: [PR] Fixes int overflow for num values [pinot]

2024-11-11 Thread via GitHub
Harnoor7 closed pull request #14423: Fixes int overflow for num values URL: https://github.com/apache/pinot/pull/14423 -- 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 unsubs

Re: [PR] Fixes int overflow for num values [pinot]

2024-11-11 Thread via GitHub
Harnoor7 commented on PR #14423: URL: https://github.com/apache/pinot/pull/14423#issuecomment-2468014321 Pls ignore this PR. -- 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.

[PR] Fixes int overflow for num values [pinot]

2024-11-11 Thread via GitHub
Harnoor7 opened a new pull request, #14423: URL: https://github.com/apache/pinot/pull/14423 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` 6

[PR] Bump org.webjars:swagger-ui from 5.18.0 to 5.18.2 [pinot]

2024-11-11 Thread via GitHub
dependabot[bot] opened a new pull request, #14422: URL: https://github.com/apache/pinot/pull/14422 Bumps [org.webjars:swagger-ui](https://github.com/swagger-api/swagger-ui) from 5.18.0 to 5.18.2. Release notes Sourced from https://github.com/swagger-api/swagger-ui/releases";>org.we

Re: [PR] Bump software.amazon.awssdk:bom from 2.29.5 to 2.29.8 [pinot]

2024-11-11 Thread via GitHub
dependabot[bot] closed pull request #14417: Bump software.amazon.awssdk:bom from 2.29.5 to 2.29.8 URL: https://github.com/apache/pinot/pull/14417 -- 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

Re: [PR] Bump org.webjars:swagger-ui from 5.18.0 to 5.18.1 [pinot]

2024-11-11 Thread via GitHub
dependabot[bot] commented on PR #14408: URL: https://github.com/apache/pinot/pull/14408#issuecomment-2467929414 Superseded by #14422. -- 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

[PR] Bump com.nimbusds:nimbus-jose-jwt from 9.45 to 9.46 [pinot]

2024-11-11 Thread via GitHub
dependabot[bot] opened a new pull request, #14421: URL: https://github.com/apache/pinot/pull/14421 Bumps [com.nimbusds:nimbus-jose-jwt](https://bitbucket.org/connect2id/nimbus-jose-jwt) from 9.45 to 9.46. Changelog Sourced from https://bitbucket.org/connect2id/nimbus-jose-jwt/src/

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.29.8 deleted (was 6f619372ff)

2024-11-11 Thread github-bot
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.29.8 in repository https://gitbox.apache.org/repos/asf/pinot.git was 6f619372ff Bump software.amazon.awssdk:bom from 2.29.5 to 2.29.8 The

[PR] Bump software.amazon.awssdk:bom from 2.29.5 to 2.29.9 [pinot]

2024-11-11 Thread via GitHub
dependabot[bot] opened a new pull request, #14420: URL: https://github.com/apache/pinot/pull/14420 Bumps software.amazon.awssdk:bom from 2.29.5 to 2.29.9. Most Recent Ignore Conditions Applied to This Pull Request | Dependency Name | Ignore Conditions | | --- | --- | |

(pinot) branch dependabot/maven/org.webjars-swagger-ui-5.18.2 created (now a10bf2aa99)

2024-11-11 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.webjars-swagger-ui-5.18.2 in repository https://gitbox.apache.org/repos/asf/pinot.git at a10bf2aa99 Bump org.webjars:swagger-ui from 5.18.0 to 5.18.2 No new revis

(pinot) branch dependabot/maven/org.webjars-swagger-ui-5.18.1 deleted (was 5e18925037)

2024-11-11 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.webjars-swagger-ui-5.18.1 in repository https://gitbox.apache.org/repos/asf/pinot.git was 5e18925037 Bump org.webjars:swagger-ui from 5.18.0 to 5.18.1 The revision

Re: [PR] Bump org.webjars:swagger-ui from 5.18.0 to 5.18.1 [pinot]

2024-11-11 Thread via GitHub
dependabot[bot] closed pull request #14408: Bump org.webjars:swagger-ui from 5.18.0 to 5.18.1 URL: https://github.com/apache/pinot/pull/14408 -- 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 spec

(pinot) branch dependabot/maven/com.nimbusds-nimbus-jose-jwt-9.46 created (now f66d6f0318)

2024-11-11 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/com.nimbusds-nimbus-jose-jwt-9.46 in repository https://gitbox.apache.org/repos/asf/pinot.git at f66d6f0318 Bump com.nimbusds:nimbus-jose-jwt from 9.45 to 9.46 No new

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.29.9 created (now 495d6725ee)

2024-11-11 Thread github-bot
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.29.9 in repository https://gitbox.apache.org/repos/asf/pinot.git at 495d6725ee Bump software.amazon.awssdk:bom from 2.29.5 to 2.29.9 No n

Re: [PR] Bump software.amazon.awssdk:bom from 2.29.5 to 2.29.8 [pinot]

2024-11-11 Thread via GitHub
dependabot[bot] commented on PR #14417: URL: https://github.com/apache/pinot/pull/14417#issuecomment-2467927333 Superseded by #14420. -- 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

Re: [PR] Use Picocli to generate help [pinot]

2024-11-11 Thread via GitHub
codecov-commenter commented on PR #14419: URL: https://github.com/apache/pinot/pull/14419#issuecomment-2467515808 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/14419?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

Re: [PR] Fix long value parsing in jsonextractscalar [pinot]

2024-11-11 Thread via GitHub
gortiz commented on code in PR #14337: URL: https://github.com/apache/pinot/pull/14337#discussion_r1836107425 ## pinot-core/src/main/java/org/apache/pinot/core/util/NumberUtils.java: ## @@ -0,0 +1,224 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one + * or

(pinot) branch master updated: Remove Refactor TODO (#14338)

2024-11-11 Thread gortiz
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 46394c24d6 Remove Refactor TODO (#14338) 46394c24

Re: [PR] Remove Refactor TODO [pinot]

2024-11-11 Thread via GitHub
gortiz merged PR #14338: URL: https://github.com/apache/pinot/pull/14338 -- 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

Re: [I] Cleanup `String.format()` usage [pinot]

2024-11-11 Thread via GitHub
gortiz commented on issue #14404: URL: https://github.com/apache/pinot/issues/14404#issuecomment-2467469252 > as it is slightly more readable I think that is very subjective. People coming from C background may find it more readable, but people coming from more modern languages where