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

Merge pull request #497 from metamx/fix-metrics 

Fix server metrics to avoid NaN and Infinity
This commit is contained in:
xvrl 2014-04-22 10:58:42 -07:00
parent bed8860fd4 cde93d810c
commit 4e1c159ff1
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
server/src/main/java/io/druid/server/metrics/ServerMonitor.java
View File

@ -59,7 +59,8 @@ public class ServerMonitor extends AbstractMonitor
.setUser2(serverConfig.getTier());
emitter.emit(builder.build("server/segment/used", used));
emitter.emit(builder.build("server/segment/usedPercent", used / (double) serverConfig.getMaxSize()));
final double usedPercent = serverConfig.getMaxSize() == 0 ? 0 : used / (double) serverConfig.getMaxSize();
emitter.emit(builder.build("server/segment/usedPercent", usedPercent));
}
for (Map.Entry<String, Long> entry : serverManager.getDataSourceCounts().entrySet()) {
@ -74,7 +75,8 @@ public class ServerMonitor extends AbstractMonitor
final ServiceMetricEvent.Builder builder = new ServiceMetricEvent.Builder().setUser2(serverConfig.getTier());
emitter.emit(builder.build("server/segment/totalUsed", totalUsed));
emitter.emit(builder.build("server/segment/totalUsedPercent", totalUsed / (double) serverConfig.getMaxSize()));
final double totalUsedPercent = serverConfig.getMaxSize() == 0 ? 0 : totalUsed / (double) serverConfig.getMaxSize();
emitter.emit(builder.build("server/segment/totalUsedPercent", totalUsedPercent));
emitter.emit(builder.build("server/segment/totalCount", totalCount));
return true;