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

fix server metrics to avoid NaN and Infinity

This commit is contained in:
fjy 2014-04-22 10:26:46 -07:00
parent bed8860fd4
commit cde93d810c
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()); .setUser2(serverConfig.getTier());
emitter.emit(builder.build("server/segment/used", used)); 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()) { 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()); final ServiceMetricEvent.Builder builder = new ServiceMetricEvent.Builder().setUser2(serverConfig.getTier());
emitter.emit(builder.build("server/segment/totalUsed", totalUsed)); 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)); emitter.emit(builder.build("server/segment/totalCount", totalCount));
return true; return true;