Github user bitblender commented on a diff in the pull request:
https://github.com/apache/drill/pull/921#discussion_r148674083
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/coord/ClusterCoordinator.java
---
@@ -60,7 +61,26 @@
*/
public abstract Collection<DrillbitEndpoint> getAvailableEndpoints();
+ /**
+ * Get a collection of ONLINE drillbit endpoints by excluding the
drillbits
+ * that are in QUIESCENT state (drillbits that are shutting down).
Primarily used by the planner
+ * to plan queries only on ONLINE drillbits and used by the client
during initial connection
+ * phase to connect to a drillbit (foreman)
+ * @return A collection of ONLINE endpoints
+ */
+
+ public abstract Collection<DrillbitEndpoint> getOnlineEndPoints();
+
+ public abstract RegistrationHandle update(RegistrationHandle handle,
State state);
+
public interface RegistrationHandle {
+ /**
+ * Get the drillbit endpoint associated with the registration handle
+ * @return drillbit endpoint
+ */
+ public abstract DrillbitEndpoint getEndPoint();
+
+ public abstract void setEndPoint( DrillbitEndpoint endpoint);
--- End diff --
spacing
---
