[GitHub] [drill] jnturton commented on a diff in pull request #2650: DRILL-8307: Ensure thread safety in the Druid plugin HTTP client

jnturton commented on code in PR #2650: URL: https://github.com/apache/drill/pull/2650#discussion_r973710035 ## contrib/storage-druid/src/main/java/org/apache/drill/exec/store/druid/rest/RestClientWrapper.java: ## @@ -17,38 +17,36 @@ */ package

[GitHub] [drill] jnturton commented on a diff in pull request #2650: DRILL-8307: Ensure thread safety in the Druid plugin HTTP client

jnturton commented on code in PR #2650: URL: https://github.com/apache/drill/pull/2650#discussion_r973688571 ## contrib/storage-druid/src/main/java/org/apache/drill/exec/store/druid/rest/RestClientWrapper.java: ## @@ -17,38 +17,36 @@ */ package

[GitHub] [drill] jnturton commented on a diff in pull request #2650: DRILL-8307: Ensure thread safety in the Druid plugin HTTP client

jnturton commented on code in PR #2650: URL: https://github.com/apache/drill/pull/2650#discussion_r973543624 ## contrib/storage-druid/src/main/java/org/apache/drill/exec/store/druid/rest/DruidQueryClient.java: ## @@ -48,20 +48,21 @@ public DruidQueryClient(String brokerURI,

[GitHub] [drill] jnturton commented on a diff in pull request #2650: DRILL-8307: Ensure thread safety in the Druid plugin HTTP client

jnturton commented on code in PR #2650: URL: https://github.com/apache/drill/pull/2650#discussion_r973144234 ## contrib/storage-druid/src/main/java/org/apache/drill/exec/store/druid/rest/DruidQueryClient.java: ## @@ -48,20 +48,21 @@ public DruidQueryClient(String brokerURI,

[GitHub] [drill] jnturton commented on a diff in pull request #2650: DRILL-8307: Ensure thread safety in the Druid plugin HTTP client

jnturton commented on code in PR #2650: URL: https://github.com/apache/drill/pull/2650#discussion_r973144234 ## contrib/storage-druid/src/main/java/org/apache/drill/exec/store/druid/rest/DruidQueryClient.java: ## @@ -48,20 +48,21 @@ public DruidQueryClient(String brokerURI,