Yelp / mrjob

Run MapReduce jobs on Hadoop or Amazon Web Services
http://packages.python.org/mrjob/
Other
2.62k stars 586 forks source link

useless return value from make_pooled_cluster() in pooling tests #2196

Closed coyotemarin closed 3 years ago

coyotemarin commented 4 years ago

make_pooled_cluster() in tests/test_emr_pooling.py returns a runner and a cluster ID, but the runner is never used. We should just return the cluster ID.

This is a very simple fix (and self-testing), but it should be a separate pull request, so as not to obscure more substantive changes.