Apache
/
druid
mirror of https://github.com/apache/druid.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

address cr

This commit is contained in:
fjy 2014-06-19 16:48:43 -07:00
parent b313601a3b
commit 2aab2a0cd9
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
server/src/main/java/io/druid/segment/loading/SegmentLoaderConfig.java
View File

@ -38,7 +38,7 @@ public class SegmentLoaderConfig
private boolean deleteOnRemove = true;
@JsonProperty("dropSegmentDelayMillis")
private int dropSegmentDelayMillis = 0;
private int dropSegmentDelayMillis = 5 * 60 * 1000; // 5 mins
@JsonProperty
private File infoDir = null;

6
server/src/main/java/io/druid/server/coordination/ZkCoordinator.java
View File

@ -233,7 +233,7 @@ public class ZkCoordinator extends BaseZkCoordinator
@Override
public void removeSegment(final DataSegment segment, DataSegmentChangeCallback callback)
public void removeSegment(final DataSegment segment, final DataSegmentChangeCallback callback)
{
try {
File segmentInfoCacheFile = new File(config.getInfoDir(), segment.getIdentifier());
@ -243,6 +243,7 @@ public class ZkCoordinator extends BaseZkCoordinator
announcer.unannounceSegment(segment);
log.info("Completely removing [%s] in [%,d] millis", segment.getIdentifier(), config.getDropSegmentDelayMillis());
exec.schedule(
new Runnable()
{
@ -253,6 +254,9 @@ public class ZkCoordinator extends BaseZkCoordinator
serverManager.dropSegment(segment);
}
catch (Exception e) {
log.makeAlert(e, "Failed to remove segment! Possible resource leak!")
.addData("segment", segment)
.emit();
throw Throwables.propagate(e);
}
}