You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@livy.apache.org by mgaido91 <gi...@git.apache.org> on 2018/11/27 13:32:00 UTC

[GitHub] incubator-livy pull request #130: [LIVY-537] Add hiveconf for setting the nu...

GitHub user mgaido91 opened a pull request:

    https://github.com/apache/incubator-livy/pull/130

    [LIVY-537] Add hiveconf for setting the num of executors in thrift sessions

    ## 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.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/mgaido91/incubator-livy LIVY-537

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-livy/pull/130.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #130
    
----
commit 105f79ac33b2c5e22212e88f9e7a68d0ff1a2647
Author: Marco Gaido <mg...@...>
Date:   2018-11-27T13:30:15Z

    [LIVY-537] Add hiveconf for setting the num of executors in thrift sessions

----


---

[GitHub] incubator-livy issue #130: [LIVY-537] Add hiveconf for setting the num of ex...

Posted by mgaido91 <gi...@git.apache.org>.
Github user mgaido91 commented on the issue:

    https://github.com/apache/incubator-livy/pull/130
  
    cc @vanzin @jerryshao 


---

[GitHub] incubator-livy issue #130: [LIVY-537] Add hiveconf for setting the num of ex...

Posted by vanzin <gi...@git.apache.org>.
Github user vanzin commented on the issue:

    https://github.com/apache/incubator-livy/pull/130
  
    Merging to master.


---

[GitHub] incubator-livy pull request #130: [LIVY-537] Add hiveconf for setting the nu...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-livy/pull/130


---