[I] Convert BYTES DataType to DECIMAL in Apache Pinot [pinot]

2024-11-20 Thread via GitHub
rajat-sr1704 opened a new issue, #14510: URL: https://github.com/apache/pinot/issues/14510 I am working on ingesting realtime data from apache kafka topic. The data is stored in Avro format with Schema Registry, I want to ingest the real time data into apache pinot. As the data in Avro is i

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
yashmayya commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1851411034 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/plannode/MailboxReceiveNode.java: ## @@ -93,6 +96,10 @@ public MailboxSendNode getSender() {

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.29.17 deleted (was 66c4f32b50)

2024-11-20 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya pushed a change to branch dependabot/maven/software.amazon.awssdk-bom-2.29.17 in repository https://gitbox.apache.org/repos/asf/pinot.git was 66c4f32b50 Bump software.amazon.awssdk:bom from 2.29.16 to 2.29.17 Th

(pinot) branch master updated (fe7d1d9a6e -> 81f16f3056)

2024-11-20 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from fe7d1d9a6e Updating realtime table to be in the metric key (#14509) add 81f16f3056 Bump software.amazon.awssdk:

Re: [PR] Bump software.amazon.awssdk:bom from 2.29.16 to 2.29.17 [pinot]

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

Re: [PR] Fix crc mismatch during deepstore upload retry task [pinot]

2024-11-20 Thread via GitHub
ankitsultana commented on code in PR #14506: URL: https://github.com/apache/pinot/pull/14506#discussion_r1851302322 ## pinot-server/src/main/java/org/apache/pinot/server/api/resources/TablesResource.java: ## @@ -781,6 +781,7 @@ public String uploadLLCSegment( String realt

Re: [PR] Fix crc mismatch during deepstore upload retry task [pinot]

2024-11-20 Thread via GitHub
ankitsultana commented on code in PR #14506: URL: https://github.com/apache/pinot/pull/14506#discussion_r1851300245 ## pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/realtime/PinotLLCRealtimeSegmentManager.java: ## @@ -1555,26 +1555,38 @@ public void uploa

Re: [PR] Support for S3A Connector #14312 [pinot]

2024-11-20 Thread via GitHub
chrajeshbabu commented on PR #14474: URL: https://github.com/apache/pinot/pull/14474#issuecomment-2489911862 > Thanks for the PR. Would you please add description section with details? Thank you @mayankshriv for review. Added the description. -- This is an automated message from th

(pinot) branch master updated (6f7ed764de -> fe7d1d9a6e)

2024-11-20 Thread mcvsubbu
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 6f7ed764de Bump commons-io:commons-io from 2.11.0 to 2.18.0 (#14505) add fe7d1d9a6e Updating realtime table to b

Re: [PR] Updating realtime table to be in the metric key [pinot]

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

Re: [PR] Make TableDataManagerProvider pluggable [pinot]

2024-11-20 Thread via GitHub
Jackie-Jiang merged PR #14470: URL: https://github.com/apache/pinot/pull/14470 -- 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] Handle `IS NULL/IS NOT NULL` filtering when ExpressionTransformer outputs `null` [pinot]

2024-11-20 Thread via GitHub
itschrispeck closed issue #14433: Handle `IS NULL/IS NOT NULL` filtering when ExpressionTransformer outputs `null` URL: https://github.com/apache/pinot/issues/14433 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] Updating realtime table to be in the metric key [pinot]

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

Re: [PR] Support for S3A Connector #14312 [pinot]

2024-11-20 Thread via GitHub
chrajeshbabu commented on code in PR #14474: URL: https://github.com/apache/pinot/pull/14474#discussion_r1851151881 ## pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java: ## @@ -501,12 +504,14 @@ public long length(URI fileUr

Re: [PR] Support for S3A Connector #14312 [pinot]

2024-11-20 Thread via GitHub
chrajeshbabu commented on code in PR #14474: URL: https://github.com/apache/pinot/pull/14474#discussion_r1851151881 ## pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java: ## @@ -501,12 +504,14 @@ public long length(URI fileUr

Re: [PR] Support for S3A Connector #14312 [pinot]

2024-11-20 Thread via GitHub
chrajeshbabu commented on code in PR #14474: URL: https://github.com/apache/pinot/pull/14474#discussion_r1851151881 ## pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java: ## @@ -501,12 +504,14 @@ public long length(URI fileUr

Re: [PR] [timeseries] Response Size Limit, Metrics and Series Limit [pinot]

2024-11-20 Thread via GitHub
ankitsultana commented on code in PR #14501: URL: https://github.com/apache/pinot/pull/14501#discussion_r1851149881 ## pinot-query-runtime/src/main/java/org/apache/pinot/query/service/dispatch/timeseries/TimeSeriesDispatchClient.java: ## @@ -29,16 +29,19 @@ /** * Dispatch c

(pinot) branch dependabot/maven/commons-io-commons-io-2.18.0 deleted (was cde1c884e3)

2024-11-20 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/commons-io-commons-io-2.18.0 in repository https://gitbox.apache.org/repos/asf/pinot.git was cde1c884e3 Revert version in test The revisions that were on this branch are st

(pinot) branch master updated (15eb120701 -> 6f7ed764de)

2024-11-20 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from 15eb120701 Add metrics for bytes ingested and bytes dropped. (#14496) add 6f7ed764de Bump commons-io:commons-io fr

Re: [PR] Bump commons-io:commons-io from 2.11.0 to 2.18.0 [pinot]

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

(pinot) branch dependabot/maven/commons-io-commons-io-2.18.0 updated (377bf9d0bf -> cde1c884e3)

2024-11-20 Thread jackie
This is an automated email from the ASF dual-hosted git repository. jackie pushed a change to branch dependabot/maven/commons-io-commons-io-2.18.0 in repository https://gitbox.apache.org/repos/asf/pinot.git from 377bf9d0bf Bump commons-io:commons-io from 2.11.0 to 2.18.0 add cde1c884e3

Re: [PR] Bump commons-io:commons-io from 2.11.0 to 2.18.0 [pinot]

2024-11-20 Thread via GitHub
Jackie-Jiang commented on code in PR #14505: URL: https://github.com/apache/pinot/pull/14505#discussion_r1851133935 ## pinot-spi/pom.xml: ## @@ -78,7 +78,7 @@ commons-io commons-io - 2.11.0 +

Re: [PR] Add metrics for bytes ingested and bytes dropped. [pinot]

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

(pinot) branch master updated: Add metrics for bytes ingested and bytes dropped. (#14496)

2024-11-20 Thread lqc
This is an automated email from the ASF dual-hosted git repository. lqc 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 15eb120701 Add metrics for bytes ingested and bytes

(pinot) branch master updated: Realtime segment size threshold metrics (#14485)

2024-11-20 Thread mcvsubbu
This is an automated email from the ASF dual-hosted git repository. mcvsubbu 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 b9965c8a1c Realtime segment size threshold metr

[PR] Updating realtime table to be in the metric key [pinot]

2024-11-20 Thread via GitHub
mcvsubbu opened a new pull request, #14509: URL: https://github.com/apache/pinot/pull/14509 Based on the feedback -- 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 uns

Re: [PR] Add metrics for bytes ingested and bytes dropped. [pinot]

2024-11-20 Thread via GitHub
deemoliu commented on code in PR #14496: URL: https://github.com/apache/pinot/pull/14496#discussion_r1850979779 ## pinot-plugins/pinot-stream-ingestion/pinot-kinesis/src/main/java/org/apache/pinot/plugin/stream/kinesis/KinesisConsumer.java: ## @@ -150,8 +150,8 @@ private BytesSt

Re: [PR] Add metrics for bytes ingested and bytes dropped. [pinot]

2024-11-20 Thread via GitHub
jackluo923 commented on code in PR #14496: URL: https://github.com/apache/pinot/pull/14496#discussion_r1850974494 ## pinot-plugins/pinot-stream-ingestion/pinot-kinesis/src/main/java/org/apache/pinot/plugin/stream/kinesis/KinesisConsumer.java: ## @@ -150,8 +150,8 @@ private Bytes

Re: [PR] Allow reload for UploadedRealtimeSegmentName segments [pinot]

2024-11-20 Thread via GitHub
tibrewalpratik17 commented on code in PR #14494: URL: https://github.com/apache/pinot/pull/14494#discussion_r1850938515 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java: ## @@ -425,13 +426,14 @@ public SuccessResponse r

Re: [PR] Allow reload for UploadedRealtimeSegmentName segments [pinot]

2024-11-20 Thread via GitHub
klsince commented on code in PR #14494: URL: https://github.com/apache/pinot/pull/14494#discussion_r1850921142 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java: ## @@ -425,13 +426,14 @@ public SuccessResponse reloadSegm

Re: [PR] Allow reload for UploadedRealtimeSegmentName segments [pinot]

2024-11-20 Thread via GitHub
klsince commented on code in PR #14494: URL: https://github.com/apache/pinot/pull/14494#discussion_r1850921142 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java: ## @@ -425,13 +426,14 @@ public SuccessResponse reloadSegm

Re: [PR] Allow reload for UploadedRealtimeSegmentName segments [pinot]

2024-11-20 Thread via GitHub
tibrewalpratik17 commented on code in PR #14494: URL: https://github.com/apache/pinot/pull/14494#discussion_r1850915996 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java: ## @@ -425,13 +426,14 @@ public SuccessResponse r

Re: [PR] Add support for storing non-CLP-encodable values in a separate column; Replace CLP row value that are too large to store in FixedByteMVMutableForwardIndex with an error message. [pinot]

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

(pinot) branch master updated: Retain boolean type from CLPLogRecordExtractor decoder. (#14497)

2024-11-20 Thread lqc
This is an automated email from the ASF dual-hosted git repository. lqc 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 96bf1831b5 Retain boolean type from CLPLogRecordExtr

(pinot) branch master updated (96bf1831b5 -> cf1cf9c00f)

2024-11-20 Thread lqc
This is an automated email from the ASF dual-hosted git repository. lqc pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from 96bf1831b5 Retain boolean type from CLPLogRecordExtractor decoder. (#14497) add cf1cf9c00f Add support for storing n

Re: [PR] Retain boolean type from CLPLogRecordExtractor decoder. [pinot]

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

Re: [PR] Minion Task to support automatic Segment Refresh [pinot]

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

(pinot) branch master updated: Minion Task to support automatic Segment Refresh (#14300)

2024-11-20 Thread vvivekiyer
This is an automated email from the ASF dual-hosted git repository. vvivekiyer 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 a9abd143f0 Minion Task to support automatic S

Re: [PR] [timeseries] Response Size Limit, Metrics and Series Limit [pinot]

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

Re: [PR] [Spool] Actual implementation [pinot]

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

Re: [PR] Allow reload for UploadedRealtimeSegmentName segments [pinot]

2024-11-20 Thread via GitHub
klsince commented on code in PR #14494: URL: https://github.com/apache/pinot/pull/14494#discussion_r1850747333 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java: ## @@ -425,13 +426,14 @@ public SuccessResponse reloadSegm

Re: [PR] Bug fix: Do not ignore scheme property in Java client [pinot]

2024-11-20 Thread via GitHub
cyrilou242 commented on code in PR #12332: URL: https://github.com/apache/pinot/pull/12332#discussion_r1849438185 ## pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/JsonAsyncHttpPinotClientTransportFactory.java: ## @@ -85,8 +85,9 @@ public JsonAsyncHttpPino

Re: [PR] Apply filter reduce expressions Calcite rule at the end to prevent literal-only filter pushdown to leaf stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14448: URL: https://github.com/apache/pinot/pull/14448#discussion_r1848412398 ## pinot-query-planner/src/test/resources/queries/LiteralEvaluationPlans.json: ## @@ -257,6 +257,55 @@ "description": "select non-exist literal function",

Re: [PR] Allow reload for UploadedRealtimeSegmentName segments [pinot]

2024-11-20 Thread via GitHub
rohityadav1993 commented on code in PR #14494: URL: https://github.com/apache/pinot/pull/14494#discussion_r1850381910 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java: ## @@ -425,13 +426,14 @@ public SuccessResponse rel

Re: [I] SQL Date functions not available [pinot]

2024-11-20 Thread via GitHub
elangovankrishna commented on issue #4835: URL: https://github.com/apache/pinot/issues/4835#issuecomment-2489067731 bump up -- 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.

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1850270296 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/plannode/MailboxSendNode.java: ## @@ -50,8 +53,42 @@ public MailboxSendNode(int stageId, DataSchema d

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1850602515 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/plannode/MailboxSendNode.java: ## @@ -50,8 +53,42 @@ public MailboxSendNode(int stageId, DataSchema d

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1850587411 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/plannode/MailboxSendNode.java: ## @@ -50,8 +53,42 @@ public MailboxSendNode(int stageId, DataSchema d

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1850580654 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/logical/EquivalentStagesReplacer.java: ## @@ -0,0 +1,79 @@ +/** + * Licensed to the Apache Software F

[PR] restrict UploadedRealtimeSegmentName upload to realtime table [pinot]

2024-11-20 Thread via GitHub
rohityadav1993 opened a new pull request, #14508: URL: https://github.com/apache/pinot/pull/14508 `backward-incompat` Adds a validation to not allow UploadedRealtimeSegmentName segments to offline tables. Though a backward incompatible change but the documentations in UploadedRealtim

Re: [PR] Allow reload for UploadedRealtimeSegmentName segments [pinot]

2024-11-20 Thread via GitHub
rohityadav1993 commented on code in PR #14494: URL: https://github.com/apache/pinot/pull/14494#discussion_r1850516862 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java: ## @@ -425,13 +426,14 @@ public SuccessResponse rel

Re: [PR] Allow reload for UploadedRealtimeSegmentName segments [pinot]

2024-11-20 Thread via GitHub
rohityadav1993 commented on code in PR #14494: URL: https://github.com/apache/pinot/pull/14494#discussion_r1850381910 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotSegmentRestletResource.java: ## @@ -425,13 +426,14 @@ public SuccessResponse rel

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1850344673 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/plannode/MailboxSendNode.java: ## @@ -28,7 +30,7 @@ public class MailboxSendNode extends BasePla

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
yashmayya commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1850339957 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/plannode/MailboxSendNode.java: ## @@ -28,7 +30,7 @@ public class MailboxSendNode extends Base

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1850256883 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/plannode/MailboxSendNode.java: ## @@ -28,7 +30,7 @@ public class MailboxSendNode extends BasePla

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1850244343 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/plannode/MailboxSendNode.java: ## @@ -50,8 +53,42 @@ public MailboxSendNode(int stageId, DataSchema d

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1850241625 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/logical/EquivalentStagesReplacer.java: ## @@ -0,0 +1,79 @@ +/** + * Licensed to the Apache Software F

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1850238718 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/logical/PlanFragmenter.java: ## @@ -150,6 +150,7 @@ public PlanNode visitExchange(ExchangeNode node,

[PR] Fix crc mismatch during deepstore upload retry task [pinot]

2024-11-20 Thread via GitHub
tibrewalpratik17 opened a new pull request, #14506: URL: https://github.com/apache/pinot/pull/14506 After https://github.com/apache/pinot/pull/14406, we are able to successfully take deepstore backup but now we see that there are lot of UpsertCompactionTask failures with the following error

Re: [PR] Stops consumption if Num Of Values per segment column crosses a threshold [pinot]

2024-11-20 Thread via GitHub
Harnoor7 commented on code in PR #14479: URL: https://github.com/apache/pinot/pull/14479#discussion_r1850202734 ## pinot-common/src/main/java/org/apache/pinot/common/protocols/SegmentCompletionProtocol.java: ## @@ -149,6 +149,7 @@ public enum ControllerResponseStatus { public

Re: [PR] Stops consumption if Num Of Values per segment column crosses a threshold [pinot]

2024-11-20 Thread via GitHub
Harnoor7 commented on code in PR #14479: URL: https://github.com/apache/pinot/pull/14479#discussion_r1850200334 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/indexsegment/mutable/MutableSegmentImpl.java: ## @@ -797,6 +801,18 @@ private void addNewRow(int do

(pinot) branch master updated (2a9ddf3677 -> c964d7c1fd)

2024-11-20 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git from 2a9ddf3677 Bump software.amazon.awssdk:bom from 2.29.15 to 2.29.16 (#14491) add c964d7c1fd Bump com.google.err

(pinot) branch dependabot/maven/commons-io-commons-io-2.18.0 created (now 377bf9d0bf)

2024-11-20 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/commons-io-commons-io-2.18.0 in repository https://gitbox.apache.org/repos/asf/pinot.git at 377bf9d0bf Bump commons-io:commons-io from 2.11.0 to 2.18.0 No new revisio

[PR] Bump commons-io:commons-io from 2.11.0 to 2.18.0 [pinot]

2024-11-20 Thread via GitHub
dependabot[bot] opened a new pull request, #14505: URL: https://github.com/apache/pinot/pull/14505 Bumps commons-io:commons-io from 2.11.0 to 2.18.0. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=commons-io:co

[PR] Bump software.amazon.awssdk:bom from 2.29.16 to 2.29.17 [pinot]

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

(pinot) branch dependabot/maven/software.amazon.awssdk-bom-2.29.17 created (now 66c4f32b50)

2024-11-20 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.17 in repository https://gitbox.apache.org/repos/asf/pinot.git at 66c4f32b50 Bump software.amazon.awssdk:bom from 2.29.16 to 2.29.17 N

Re: [I] Support new data type: `TimestampWithTimeZone` [pinot]

2024-11-20 Thread via GitHub
mghildiy commented on issue #13004: URL: https://github.com/apache/pinot/issues/13004#issuecomment-2488123261 I have a query. User input would be something like '2021-01-01 01:01:01.001 America/Chicago', and not with offsets. Am I right? -- This is an automated message from the Apache Git

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
yashmayya commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1849912468 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/plannode/MailboxReceiveNode.java: ## @@ -126,7 +133,8 @@ public boolean equals(Object o) {

[PR] [WIP] Adds check if controller tmp dir exists [pinot]

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

Re: [PR] [Spool] Introduce stage replacer and change send nodes to be able to send to more than one stage [pinot]

2024-11-20 Thread via GitHub
gortiz commented on code in PR #14495: URL: https://github.com/apache/pinot/pull/14495#discussion_r1849804457 ## pinot-query-planner/src/main/java/org/apache/pinot/query/planner/plannode/MailboxSendNode.java: ## @@ -50,8 +55,50 @@ public MailboxSendNode(int stageId, DataSchema d

(pinot) branch master updated: Fix timestamp literal handling in the multi-stage query engine (#14502)

2024-11-20 Thread yashmayya
This is an automated email from the ASF dual-hosted git repository. yashmayya 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 a914940b37 Fix timestamp literal handling in t

Re: [PR] Fix timestamp literal handling in the multi-stage query engine [pinot]

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