Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/921#discussion_r150973864
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/server/DrillbitContext.java
---
@@ -157,10 +157,29 @@ public DrillConfig getConfig() {
return context.getConfig();
}
- public Collection<DrillbitEndpoint> getBits() {
+ public Collection<DrillbitEndpoint> getAvailableBits() {
return coord.getAvailableEndpoints();
}
+ public Collection<DrillbitEndpoint> getBits() {
+ return coord.getOnlineEndPoints();
+ }
+
+ public boolean isOnline(DrillbitEndpoint endpoint) { return
endpoint.getState().equals(DrillbitEndpoint.State.ONLINE); }
--- End diff --
A bit too much code for a single line. Maybe put the body inside the method
on a separate line.
---
