Github user bitblender commented on a diff in the pull request:

    https://github.com/apache/drill/pull/921#discussion_r148686859
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/Drillbit.java ---
    @@ -69,14 +73,30 @@
     
       public final static String SYSTEM_OPTIONS_NAME = 
"org.apache.drill.exec.server.Drillbit.system_options";
     
    -  private boolean isClosed = false;
    -
       private final ClusterCoordinator coord;
       private final ServiceEngine engine;
       private final PersistentStoreProvider storeProvider;
       private final WorkManager manager;
       private final BootStrapContext context;
       private final WebServer webServer;
    +  private final int gracePeriod;
    +  private DrillbitStateManager stateManager;
    +
    +  public void setQuiescentMode(boolean quiescentMode) {
    +    this.quiescentMode = quiescentMode;
    +  }
    +
    +  private volatile boolean quiescentMode;
    +
    +  public void setForceful_shutdown(boolean forceful_shutdown) {
    +    this.forceful_shutdown = forceful_shutdown;
    +  }
    +
    +  private volatile boolean forceful_shutdown = false;
    --- End diff --
    
    any reason why you did not camel-case this variable? 


---

Reply via email to