Skip to content

Commit

Permalink
[LIVY-537] Add hiveconf for setting the num of executors in thrift se…
Browse files Browse the repository at this point in the history
…ssions

## What changes were proposed in this pull request?

We missed to add a parameter to set the number of executors when creating a session through the thrift API. See https://issues.apache.org/jira/browse/LIVY-537.

## How was this patch tested?

Manual tests.

Author: Marco Gaido <[email protected]>

Closes apache#130 from mgaido91/LIVY-537.
  • Loading branch information
mgaido91 authored and Marcelo Vanzin committed Nov 28, 2018
1 parent 8504c64 commit ae2228f
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -594,6 +594,8 @@ object LivyThriftSessionManager extends Logging {
createInteractiveRequest.executorMemory = Some(value)
case "set:hiveconf:livy.session.executorCores" =>
createInteractiveRequest.executorCores = convertConfValueToInt(key, value)
case "set:hiveconf:livy.session.numExecutors" =>
createInteractiveRequest.numExecutors = convertConfValueToInt(key, value)
case "set:hiveconf:livy.session.queue" =>
createInteractiveRequest.queue = Some(value)
case "set:hiveconf:livy.session.name" =>
Expand Down

0 comments on commit ae2228f

Please sign in to comment.