Github user Ben-Zvi commented on a diff in the pull request:

    https://github.com/apache/drill/pull/958#discussion_r140938324
  
    --- Diff: 
exec/memory/base/src/main/java/org/apache/drill/exec/memory/Accountant.java ---
    @@ -80,6 +119,23 @@ public Accountant(Accountant parent, long reservation, 
long maxAllocation) {
       }
     
       /**
    +   * Request lenient allocations: allows exceeding the allocation limit
    +   * by the configured grace amount. The request is granted only if strict
    +   * limits are not required.
    +   *
    +   * @param enable
    +   */
    +  public boolean setLenient() {
    --- End diff --
    
    Why "set" returns a boolean ? Should better have a separate "get".  


---

Reply via email to