-
Notifications
You must be signed in to change notification settings - Fork 12
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
Basic command line options enhancement #3
Conversation
@@ -33,7 +33,8 @@ CTOP is a tool which allows you to quickly find out what's happening on a machin | |||
* Metrics are for one node only (not cluster-wide) | |||
|
|||
## Notes | |||
* CTOP should run on anything, but I've only tested it on Ubuntu. | |||
* CTOP should run on anything. Originally was developed on Ubuntu, runs | |||
great on CentOS and MacOS X. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we change this to: "Originally was developed on Ubuntu, and it's reported to run on CentOS and MacOS X"?
Great is relative 8)
I am not familiar MX4J so I don't know how correct is the option name change. Otherwise it looks good to me 👍 cc @chrusty |
The suggested changes look fine to me, but I'd like to keep the port called "MX4JPort" rather than "cassandraPort", because this port is opened by the MX4J plugin rather than by cassandra itself. This should remind people that they need to be running Cassandra with MX4J if they find that it doesn't work. The rest of the stuff looks good though. |
Basic command line options enhancement
@chrusty thanks! |
--port
option