snleee commented on a change in pull request #8242:
URL: https://github.com/apache/pinot/pull/8242#discussion_r813459592
##########
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/PinotHelixResourceManager.java
##########
@@ -2883,7 +2883,9 @@ public String startReplaceSegments(String
tableNameWithType, List<String> segmen
// 2. Proactively delete the oldest data snapshot to make sure that
we only keep at most 2 data snapshots
// at any time in case of REFRESH use case.
if (forceCleanup) {
- if (lineageEntry.getState() == LineageEntryState.IN_PROGRESS &&
CollectionUtils
+ // TODO: Skip conflict checks for empty segmentsFrom for now. More
conflict
+ // checks over segmentsTo may be needed and added later.
+ if (lineageEntry.getState() == LineageEntryState.IN_PROGRESS &&
!segmentsFrom.isEmpty() && CollectionUtils
Review comment:
We need to consider handling REFRESH & APPEND separately. For APPEND, we
should not revert the `segmentsFrom=empty`; however, we need to revert for
REFRESH.
--
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.apache.org
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
