address cr

This commit is contained in:
fjy 2014-08-14 14:39:08 -07:00
parent 6ba0a0df8b
commit c36ffd1e55

View File

@ -59,7 +59,6 @@ import javax.annotation.Nullable;
import java.io.File; import java.io.File;
import java.io.FilenameFilter; import java.io.FilenameFilter;
import java.io.IOException; import java.io.IOException;
import java.nio.file.Files;
import java.util.Arrays; import java.util.Arrays;
import java.util.Comparator; import java.util.Comparator;
import java.util.List; import java.util.List;
@ -321,7 +320,7 @@ public class RealtimePlumber implements Plumber
if (!isPushedMarker.exists()) { if (!isPushedMarker.exists()) {
removeSegment(sink, mergedTarget); removeSegment(sink, mergedTarget);
} else { } else {
log.info("Skipping already-merged sink: %s", sink); log.info("Already pushed sink[%s]", sink);
return; return;
} }
@ -360,7 +359,9 @@ public class RealtimePlumber implements Plumber
segmentPublisher.publishSegment(segment); segmentPublisher.publishSegment(segment);
if (!isPushedMarker.createNewFile()) { if (!isPushedMarker.createNewFile()) {
log.makeAlert("Unable to make marker file! WTF?!") log.makeAlert("Failed to create marker file for [%s]", schema.getDataSource())
.addData("interval", sink.getInterval())
.addData("partitionNum", segment.getShardSpec().getPartitionNum())
.addData("marker", isPushedMarker) .addData("marker", isPushedMarker)
.emit(); .emit();
} }