Github user vrozov commented on a diff in the pull request:
https://github.com/apache/drill/pull/1145#discussion_r173236107
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/client/DrillClient.java ---
@@ -371,17 +376,20 @@ protected void afterExecute(final Runnable r, final
Throwable t) {
while (triedEndpointIndex < connectTriesVal) {
endpoint = endpoints.get(triedEndpointIndex);
+
+ // Set in both props and properties since props is passed to
UserClient
+ if (!properties.containsKey(DrillProperties.SERVICE_HOST)) {
--- End diff --
- Use `put` and add `TODO` comment.
- What is a reason not to use API available in the minimum supported
version assuming that support for JDK 7 is dropped as part of
https://issues.apache.org/jira/browse/DRILL-1491?
---
