JohnTortugo commented on pull request #6287: URL: https://github.com/apache/incubator-pinot/pull/6287#issuecomment-742821590
Hi @mayankshriv @mcvsubbu - after your reviews (and talking privately on Slack) I understand that segment creation execution time isn't a pain point; memory consumption is the real pain point of this part of Pinot. Nonetheless, the proposed change helps reduce segment creation wall time. My question then is: do you think the contribution of this PR is valuable and, if so, what the next step forward is, or should I just close the PR? @mcvsubbu - The amount of time allotted for me to work on Pinot this quarter is almost over; I can't take over your work for reducing memory allocation at this time. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
