Skip to content

Commit

Permalink
Merge pull request #1734 from davidmarin/setup-sudo
Browse files Browse the repository at this point in the history
docs no longer tell you to use sudo in setup commands (fixes #1716)
  • Loading branch information
David Marin committed Feb 23, 2018
2 parents d93f649 + f31b968 commit 96dc2f3
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions docs/guides/emr-bootstrap-cookbook.rst
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,9 @@ part of your general production environment, and :mrjob-opt:`setup`
for things that are specific to your particular job. This makes things
work as expected if you are using :ref:`cluster-pooling`.

All these examples use :mrjob-opt:`bootstrap`. Not saying it's a good idea, but
all these examples will work with :mrjob-opt:`setup` as well (yes, Hadoop
tasks on EMR apparently have access to :command:`sudo`).

EMR will generally not allow you to use :command:`sudo` in
:mrjob-opt:`setup` commands. See :doc:`setup-cookbook` for how to install
libraries, etc. without using :command:`sudo`.

.. _using-pip:

Expand Down

0 comments on commit 96dc2f3

Please sign in to comment.