milenkovicm commented on code in PR #1506:
URL:
https://github.com/apache/datafusion-ballista/pull/1506#discussion_r2936787610
##########
ballista/scheduler/src/config.rs:
##########
@@ -190,6 +190,11 @@ pub struct Config {
help = "The interval to check expired or dead executors"
)]
pub expire_dead_executor_interval_seconds: u64,
+
+ #[cfg(feature = "rest-api")]
+ /// Should the rest api be disabled
+ #[arg(long, default_value_t = false, help = "Should the rest api be
disable")]
+ pub disable_rest: bool,
Review Comment:
I agree about double negation but in this case it makes sene in my opinion
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
