Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Start/stop individual (Accumulo, Kafka, HBase, Hadoop+Yarn) services separat… #37

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mdzimmerman
Copy link
Contributor

…ely.

Currently doesn't handle dependencies between services (e.g. doesn't
check if you try to stop Hadoop before Accumulo).

…ely.

Currently doesn't handle dependencies between services (e.g. doesn't
check if you try to stop Hadoop before Accumulo).
@jahhulbert-ccri
Copy link
Contributor

@mdzimmerman i just saw this i'll try to look...prob next time best to send me an email for some reason i never check this

@jahhulbert-ccri
Copy link
Contributor

so the idea is to run cloud-local accumulo stop ? we could do cloud-local stop all and `cloudlocal stop accumulo`` looks cool i'll peek at it soon

@jahhulbert-ccri jahhulbert-ccri self-assigned this Sep 9, 2016
@mdzimmerman
Copy link
Contributor Author

yeah, this was scratching my own itch-- I was load testing Kafka and wanted to shut off the services I didn't need (Accumulo+Hadoop) without having to cut and paste out the relevant lines from the cloud-local.sh script.

@mdzimmerman mdzimmerman changed the title Start/stop individual (Accumulo, Kafka, Hadoop+Yarn) services separat… Start/stop individual (Accumulo, Kafka, HBase, Hadoop+Yarn) services separat… Sep 12, 2016
start_hbase
elif [[ $2 == 'stop' ]]; then
stop_hbase
fi
elif [[ $1 == 'stop' ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To @jahhulbert-ccri 's suggestion, can we add a (admittedly redundant) "stop all" and "start all"?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As is you could type "all" if you want, wouldn't do anything different but it'd still work as expected. Aside from that, this param eval impl may need reworked if merged with the geoserver stuff.

# 1 = enabled
# 0 = disabled
acc_enable=1
hbase_enable=0
kafka_enable=1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have no perspective on this but it does seem odd that kafka is enabled by default.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants