Spark worker cleanup added.

nchammas / flintrock

A command-line tool for launching Apache Spark clusters.

Apache License 2.0

637 stars 116 forks source link

Spark worker cleanup added. #211

Closed umit closed 6 years ago

umit commented 6 years ago

This PR makes the following changes:

I tested this PR by...

Fixes #NN. Fixes #MM.

nchammas commented 6 years ago

Hi @umitunal and thanks for making your first contribution to Flintrock!

I think instead of adding one-off configurations like this, I'd rather get the work in #202 finished up. That way, users have a well-defined way of providing their own templates to use with Flintrock.

Does that work for you?

nchammas commented 6 years ago

Closing in favor of #202. Feel free to comment or reopen if you disagree!