[HIVE-21425] Use DirectExecutorService for getInputSummary

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.2.0, 4.0.0
• Fix Version/s: 4.0.0-alpha-1
• Component/s: HiveServer2
• Labels: pull-request-available
• Assignee: David Mollitor
• Reporter: David Mollitor
• Created: 11/Mar/19 17:13
• Updated: 17/Nov/22 08:50
• Resolved: 03/Apr/19 10:23

Description

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;
      }

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.

Issue Links

• links to: GitHub Pull Request #564