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
Add a startup check to fail startup when using invalid configuration … #2995
Add a startup check to fail startup when using invalid configuration … #2995
Conversation
9e1c640
to
438ae75
Compare
@@ -616,10 +619,9 @@ public enum CassandraRelevantProperties | |||
USE_NIX_RECURSIVE_DELETE("cassandra.use_nix_recursive_delete"), | |||
/** Gossiper compute expiration timeout. Default value 3 days. */ | |||
VERY_LONG_TIME_MS("cassandra.very_long_time_ms", "259200000"), | |||
WAIT_FOR_TRACING_EVENTS_TIMEOUT_SECS("cassandra.wait_for_tracing_events_timeout_secs", "0"), | |||
WAIT_FOR_TRACING_EVENTS_TIMEOUT_SECS("cassandra.wait_for_tracing_events_timeout_secs", "0") |
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.
";" should not start a new line
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.
actually, I'll revert that to its original state
@@ -1136,6 +1150,41 @@ public static void exec(ProcessBuilder pb) throws IOException | |||
} | |||
} | |||
|
|||
public static String exec(Map<String, String> env, Duration timeout, int outBufSize, int errBufSize, String... cmd) throws IOException, TimeoutException, InterruptedException |
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.
trunk 's exec method is different from 5.0 here
no description of the function and lack of some logic
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.
yes, I forgot to push this and other changes, they should be there now
No description provided.