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

2022-09-16 Thread GitBox
cgivre commented on code in PR #2650: URL: https://github.com/apache/drill/pull/2650#discussion_r973137352 ## contrib/storage-druid/src/main/java/org/apache/drill/exec/store/druid/rest/DruidQueryClient.java: ## @@ -48,20 +48,21 @@ public DruidQueryClient(String brokerURI, RestCl

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

2022-09-17 Thread GitBox
cgivre commented on code in PR #2650: URL: https://github.com/apache/drill/pull/2650#discussion_r973642898 ## contrib/storage-druid/src/main/java/org/apache/drill/exec/store/druid/rest/DruidQueryClient.java: ## @@ -48,20 +48,21 @@ public DruidQueryClient(String brokerURI, RestCl

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

2022-09-17 Thread GitBox
cgivre commented on code in PR #2650: URL: https://github.com/apache/drill/pull/2650#discussion_r973642947 ## contrib/storage-druid/src/main/java/org/apache/drill/exec/store/druid/rest/DruidQueryClient.java: ## @@ -48,20 +48,21 @@ public DruidQueryClient(String brokerURI, RestCl