feat: Ballista Text User Interface#1436
Conversation
Initial version of the TUI app that includes: * new Cargo feature `tui` to include it as part of the CLI app * New CLI option `--tui` to start the TUI app * Header navigation (Dashboard, Jobs, Metrics) * Help popup (activate with `?` or `h`) * Dashboard view (scheduler state and executors info) * Placeholder for the Jobs and Metrics views * Yaml configuration * hardcoded defaults * OS specific config (e.g. ~/.config/ballista/config.yaml) * env variables, e.g. `BALLISTA_HTTP_TIMEOUT=3000` Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
|
@martin-g i hope you're not waiting for my review :) |
|
@milenkovicm I'll continue working on it this week! |
Same for the executors when the scheduler is up
This allows starting the TUI app several times in the CLI via the `\tui` command
This way it is always visible and becomes red when the connection to the scheduler is lost
|
sorry @martin-g your question fall off my radar. IMHO it would be great to get some kind of MVP in the PR if possible, but it's up to you |
|
I'll continue adding functionalities for the other views too (jobs & metrics)! |
|
I'll try to get logical and physical plan as part of api this week |
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
removed the wrapping |
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
The MetricsData struct has some state fields which otherwise need to be ignored Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
|
#1543 has added more task related information. This kind of information gives users ability to diagnose skewed partitions. |
Use 1,2,3,... for sorting the columns in the tables
|
I guess we could do stage related changes as a follow up, wdyt? |
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
It is up to you how much code you want to review at once :-) |
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
milenkovicm
left a comment
There was a problem hiding this comment.
Thanks @martin-g heroic effort, I like it a lot. We can add improvements later


Which issue does this PR close?
Closes #1396
Rationale for this change
See #1396
What changes are included in this PR?
Initial version of the TUI app that includes:
tuito include it as part of the CLI app--tuito start the TUI app?orh)BALLISTA_HTTP_TIMEOUT=3000Are there any user-facing changes?
New option for the CLI application, --tui that starts the TUI application