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

Posted to dev@drill.apache.org by vrozov <gi...@git.apache.org> on 2018/02/09 18:45:00 UTC

[GitHub] drill pull request #1119: DRILL-6143: Made FragmentsRunner's rpc timeout a S...

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

    https://github.com/apache/drill/pull/1119#discussion_r167312527
  
    --- Diff: exec/java-exec/src/main/resources/drill-module.conf ---
    @@ -413,6 +413,7 @@ drill.exec.options: {
         # to start at least 2 partitions then HashAgg fallbacks to this case. It can be
         # enabled by setting this flag to true. By default it's set to false such that
         # query will fail if there is not enough memory
    +    drill.exec.rpc.fragrunner.timeout: 30000,
    --- End diff --
    
    The value looks a little bit high and it needs to be moved either below `drill.exec.hashagg.fallback.enabled` or above the preceding comment, otherwise LGTM.


---