[GitHub] [drill] kkhatua commented on a change in pull request #1837: DRILL-7338: REST API calls to Drill fail due to insufficient heap memory

[GitBox <gi...@apache.org>]

kkhatua commented on a change in pull request #1837: DRILL-7338: REST API calls to Drill fail due to insufficient heap memory
URL: https://github.com/apache/drill/pull/1837#discussion_r311743001
 
 

 ##########
 File path: exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/QueryWrapper.java
 ##########
 @@ -44,7 +44,7 @@
 public class QueryWrapper {
   private static final org.slf4j.Logger logger = org.slf4j.LoggerFactory.getLogger(QueryWrapper.class);
   // Heap usage threshold/trigger to provide resiliency on web server for queries submitted via HTTP
-  private static final double HEAP_MEMORY_FAILURE_THRESHOLD = 0.85;
+  private double memoryFailureThreshold = 0.85;
 
 Review comment:
   +1

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services