Re: [PR] [Work in progress]Reduce disk footprint for segment processor framework [pinot]

2024-01-07 Thread via GitHub
aishikbh commented on code in PR #12220: URL: https://github.com/apache/pinot/pull/12220#discussion_r1444226777 ## pinot-core/src/main/java/org/apache/pinot/core/segment/processing/mapper/SegmentMapper.java: ## @@ -141,28 +149,42 @@ private Map doMap() RecordRead

Re: [PR] [Work in progress]Reduce disk footprint for segment processor framework [pinot]

2024-01-07 Thread via GitHub
aishikbh commented on code in PR #12220: URL: https://github.com/apache/pinot/pull/12220#discussion_r1444226777 ## pinot-core/src/main/java/org/apache/pinot/core/segment/processing/mapper/SegmentMapper.java: ## @@ -141,28 +149,42 @@ private Map doMap() RecordRead

Re: [PR] [MINOR] Fix upsert tenant tag override check [pinot]

2024-01-07 Thread via GitHub
codecov-commenter commented on PR #12233: URL: https://github.com/apache/pinot/pull/12233#issuecomment-1880444832 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/12233?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) R

[I] I'm curious about the sequence for a graceful shutdown between the overall components of PINOT. [pinot]

2024-01-07 Thread via GitHub
minwoo-jung opened a new issue, #12234: URL: https://github.com/apache/pinot/issues/12234 hello We are deploying/operating pinot through helm chart. # Problem We restarted helm chart to upgrade pinot, After restarting, some consuming segments of the realtime table were not

[PR] [MINOR] Fix upsert tenant tag override check [pinot]

2024-01-07 Thread via GitHub
saurabhd336 opened a new pull request, #12233: URL: https://github.com/apache/pinot/pull/12233 Allow tenant tag override when consuming and completed tags are same. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [PR] [Work in progress]Reduce disk footprint for segment processor framework [pinot]

2024-01-07 Thread via GitHub
aishikbh commented on code in PR #12220: URL: https://github.com/apache/pinot/pull/12220#discussion_r1444179819 ## pinot-core/src/main/java/org/apache/pinot/core/segment/processing/framework/SegmentProcessorFramework.java: ## @@ -142,24 +136,83 @@ public List process() priv

(pinot) branch master updated: [test] add back quickstart streaming (#12231)

2024-01-07 Thread rongr
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 07bcab80f1 [test] add back quickstart streaming (#

Re: [PR] [test] add back quick-start-streaming [pinot]

2024-01-07 Thread via GitHub
walterddr merged PR #12231: URL: https://github.com/apache/pinot/pull/12231 -- 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] [Work in progress]Reduce disk footprint for segment processor framework [pinot]

2024-01-07 Thread via GitHub
swaminathanmanish commented on code in PR #12220: URL: https://github.com/apache/pinot/pull/12220#discussion_r1444085565 ## pinot-core/src/main/java/org/apache/pinot/core/segment/processing/genericrow/FileWriter.java: ## @@ -0,0 +1,27 @@ +/** + * Licensed to the Apache Software