milenkovicm commented on code in PR #1547:
URL:
https://github.com/apache/datafusion-ballista/pull/1547#discussion_r3141773687
##########
ballista/executor/src/config.rs:
##########
@@ -142,6 +143,14 @@ pub struct Config {
help = "The heartbeat interval in seconds to the scheduler for
push-based task scheduling."
)]
pub executor_heartbeat_interval_seconds: u64,
+ /// Specifying which metrics should be collected and sent to scheduler
+ #[arg(
+ short = 'm',
+ long = "metrics",
+ default_value_t = ExecutorMetricCollectionPolicy::ProcessOnly,
Review Comment:
I believe you could use ::default() so we do not need to keep changing on
two locations
--
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]
