You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by "David Mollitor (JIRA)" <ji...@apache.org> on 2019/03/11 17:14:00 UTC

[jira] [Created] (HIVE-21425) Use newDirectExecutorService for getInputSummary

David Mollitor created HIVE-21425:
-------------------------------------

             Summary: Use newDirectExecutorService for getInputSummary
                 Key: HIVE-21425
                 URL: https://issues.apache.org/jira/browse/HIVE-21425
             Project: Hive
          Issue Type: Improvement
          Components: HiveServer2
    Affects Versions: 4.0.0, 3.2.0
            Reporter: David Mollitor


{code:java|title=Utilities.java}
      int numExecutors = getMaxExecutorsForInputListing(ctx.getConf(), pathNeedProcess.size());
      if (numExecutors > 1) {
        LOG.info("Using {} threads for getContentSummary", numExecutors);
        executor = Executors.newFixedThreadPool(numExecutors,
                new ThreadFactoryBuilder().setDaemon(true)
                        .setNameFormat("Get-Input-Summary-%d").build());
      } else {
        executor = null;
      }
{code}

https://github.com/apache/hive/blob/master/ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java#L2482-L2490

Instead of using a 'null' {{ExecutorService}}, use Guava's {{DirectExecutorService}} and remove special casing for a 'null' value.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)