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 2022/02/23 01:01:00 UTC

[GitHub] [pinot] jtao15 commented on pull request #8240: allow skip proactive cleaning segment lineages

jtao15 commented on pull request #8240:
URL: https://github.com/apache/pinot/pull/8240#issuecomment-1048359394


   Even if `forceClean = true`, we should allow creating multiple lineages with empty `SegmentsFrom` list. I think we can add one more condition `!segmentsFrom.isEmpty()` on `L2886` to handle this. Also, we need to allow creating lineage entries with the same empty `SegmentsFrom` list on `L2922`.


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org