[GitHub] [drill] luocooong commented on a diff in pull request #2610: DRILL-8268: Fix Hadoop 2 and Netty lib exclusions, REST mem limiter disabled by default

2022-07-25 Thread GitBox
luocooong commented on code in PR #2610: URL: https://github.com/apache/drill/pull/2610#discussion_r929442941 ## exec/vector/pom.xml: ## @@ -25,7 +25,7 @@ org.apache.drill.exec 2.0.0-SNAPSHOT - vector + drill-vector Review Comment: Good point. It is recomm

[GitHub] [drill] luocooong commented on a diff in pull request #2610: DRILL-8268: Fix Hadoop 2 and Netty lib exclusions, REST mem limiter disabled by default

2022-07-25 Thread GitBox
luocooong commented on code in PR #2610: URL: https://github.com/apache/drill/pull/2610#discussion_r928797183 ## exec/vector/pom.xml: ## @@ -25,7 +25,7 @@ org.apache.drill.exec 2.0.0-SNAPSHOT - vector + drill-vector Review Comment: Change is not the best thin