You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2019/07/30 18:30:34 UTC
[GitHub] [incubator-pinot] jenniferdai commented on issue #4464: Delete
extra refreshed segments after segment push
jenniferdai commented on issue #4464: Delete extra refreshed segments after segment push
URL: https://github.com/apache/incubator-pinot/pull/4464#issuecomment-516538649
Your comment seems to have disappeared. What would you like me to remove?
As the preconditions was added in your commit for multiple push and is tied with your feature, I think you probably have a better idea regarding all the edge cases you require this preconditions check, etc. It may be better for you to call out cases where that preconditions will be required as you merge subsequent prs for your feature.
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org