You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by kkhatua <gi...@git.apache.org> on 2017/12/01 01:10:07 UTC

[GitHub] drill pull request #1055: DRILL-5973 : Support injection of time-bound pause...

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

    https://github.com/apache/drill/pull/1055#discussion_r154246413
  
    --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/testing/ExecutionControlsInjector.java ---
    @@ -81,8 +83,20 @@ public void injectPause(final ExecutionControls executionControls, final String
           executionControls.lookupPauseInjection(this, desc);
     
         if (pauseInjection != null) {
    -      logger.debug("Pausing at {}", desc);
    -      pauseInjection.pause();
    +      long pauseDuration = pauseInjection.getMsPause();
    +      if ( pauseDuration > 0L) {
    --- End diff --
    
    Agreed, but the only purpose of that check is to indicate during debugging that a non-zero timed pause was injected. Hence, the check.


---