Merge pull request #1170 from metamx/fix-archive-task

break up archive task actions
This commit is contained in:
Fangjin Yang 2015-03-04 22:09:00 -08:00
commit 9850b99b3b
1 changed files with 2 additions and 10 deletions

View File

@ -89,20 +89,12 @@ public class ArchiveTask extends AbstractFixedIntervalTask
log.info("OK to archive segment: %s", unusedSegment.getIdentifier());
}
List<DataSegment> archivedSegments = Lists.newLinkedList();
// Move segments
for (DataSegment segment : unusedSegments) {
archivedSegments.add(toolbox.getDataSegmentArchiver().archive(segment));
final DataSegment archivedSegment = toolbox.getDataSegmentArchiver().archive(segment);
toolbox.getTaskActionClient().submit(new SegmentMetadataUpdateAction(ImmutableSet.of(archivedSegment)));
}
// Update metadata for moved segments
toolbox.getTaskActionClient().submit(
new SegmentMetadataUpdateAction(
ImmutableSet.copyOf(archivedSegments)
)
);
return TaskStatus.success(getId());
}
}