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 @@ <groupId>org.apache.drill.exec</groupId> <version>2.0.0-SNAPSHOT</version> </parent> - <artifactId>vector</artifactId> + <artifactId>drill-vector</artifactId> Review Comment: Change is not the best thing without more new things. ########## pom.xml: ########## @@ -3965,6 +3977,46 @@ </exclusion> </exclusions> </dependency> + <dependency> Review Comment: The dependence on Hadoop was tricky. Ensure that the dependency tree does not have a new dependency. ########## exec/java-exec/src/main/resources/drill-module.conf: ########## @@ -190,7 +190,9 @@ drill.exec: { maximum: 9223372036854775807 } }, - memory.heap.failure.threshold: 0.85, + # Default to failing queries only at 100% heap usage, i.e. the heap usage + # limiting logic in the REST API is disabled by default. + memory.heap.failure.threshold: 1.0, Review Comment: That's okay. In fact, this option is only used for REST modules and it has little effect before Streaming RESTful feature joins because we cannot expect users to send long or short queries. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org