[GitHub] [drill] jnturton commented on a diff in pull request #2655: DRILL-8314: Add support for automatically retrying and disabling broken storage plugins

jnturton commented on code in PR #2655: URL: https://github.com/apache/drill/pull/2655#discussion_r981927883 ## exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java: ## @@ -130,11 +132,18 @@ private static PhysicalPlan convertPlan(QueryContext

[GitHub] [drill] jnturton commented on a diff in pull request #2660: DRILL-8320: Prevent Infinite Pagination for Index Paginator

jnturton commented on code in PR #2660: URL: https://github.com/apache/drill/pull/2660#discussion_r981908934 ## contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/HttpBatchReader.java: ## @@ -66,7 +67,7 @@ public class HttpBatchReader implements ManagedReader

[jira] [Created] (DRILL-8320) Prevent Infinite Pagination for Index Paginator

Charles Givre created DRILL-8320: Summary: Prevent Infinite Pagination for Index Paginator Key: DRILL-8320 URL: https://issues.apache.org/jira/browse/DRILL-8320 Project: Apache Drill Issue

[GitHub] [drill] cgivre opened a new pull request, #2660: DRILL-8320: Prevent Infinite Pagination for Index Paginator

cgivre opened a new pull request, #2660: URL: https://github.com/apache/drill/pull/2660 # [DRILL-8320](https://issues.apache.org/jira/browse/DRILL-8320): Prevent Infinite Pagination for Index Paginator ## Description In some cases that use keyset/index pagination, if the API does

[GitHub] [drill] vvysotskyi commented on a diff in pull request #2655: DRILL-8314: Add support for automatically retrying and disabling broken storage plugins

vvysotskyi commented on code in PR #2655: URL: https://github.com/apache/drill/pull/2655#discussion_r981716643 ## exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/SqlHandlerConfig.java: ## @@ -52,12 +59,81 @@ public QueryContext getContext() { return

[GitHub] [drill] vvysotskyi commented on a diff in pull request #2655: DRILL-8314: Add support for automatically retrying and disabling broken storage plugins

vvysotskyi commented on code in PR #2655: URL: https://github.com/apache/drill/pull/2655#discussion_r981705289 ## exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java: ## @@ -130,11 +132,18 @@ private static PhysicalPlan convertPlan(QueryContext

[GitHub] [drill] jnturton commented on a diff in pull request #2655: DRILL-8314: Add support for automatically retrying and disabling broken storage plugins

jnturton commented on code in PR #2655: URL: https://github.com/apache/drill/pull/2655#discussion_r981328928 ## exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java: ## @@ -130,11 +132,18 @@ private static PhysicalPlan convertPlan(QueryContext