WIP: Ballista Text User Interface app#1433
Conversation
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Navigate to each one with: 'd', 'j' and 'm' keys respetively 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>
Remove obsolete functionality (search). Update the help popup window contents. 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>
|
would it make sense to switch from CLI to TUI? would it be possible to have |
| pub(crate) fn new() -> Result<Self, ConfigError> { | ||
| let config_dir = dirs::config_dir() | ||
| .unwrap_or_else(|| dirs::home_dir().unwrap().join(".config")) | ||
| .join("ballista-tui"); |
There was a problem hiding this comment.
can we keep it as ballista-cli?
There was a problem hiding this comment.
How about just ballista ?
And then use cli and/or tui as a parent node in the config.
This way the same config file could be used to provide the defaults for the whole cli+tui app and maybe even for the scheduler and executors later. AFAIS the cli app starts a scheduler and executor if host+port are not provided.
I think it should be possible! |
| _ _ _ | ||
| | \ _. _|_ _. |_ _ o _ ._ |_) _. | | o _ _|_ _. | ||
| |_/ (_| |_ (_| | |_| _> | (_) | | |_) (_| | | | _> |_ (_| | ||
| "#; |
There was a problem hiding this comment.
Could we have capital 'F' in DataFusion
There was a problem hiding this comment.
I think it is capital. Let me experiment with some other text banner
implemented with 3b289d9 |
Bumps [actions/checkout](https://github.com/actions/checkout) from 5.0.1 to 6.0.2. - [Release notes](https://github.com/actions/checkout/releases) - [Commits](actions/checkout@v5.0.1...v6.0.2) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 6.0.2 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…pache#1400) * Cluster RPC customizations to support TLS and custom headers * Add TLS support to scheduler flight proxy service - Update BallistaFlightProxyService to accept use_tls and customize_endpoint parameters - Add use_tls field to SchedulerConfig with with_use_tls() builder method - Unify EndpointOverrideFn type across crates to use ballista_core::extension definition - Update flight proxy to use https/http scheme based on TLS configuration - Apply custom endpoint configuration for TLS certificate setup
Bumps [tonic-prost-build](https://github.com/hyperium/tonic) from 0.14.2 to 0.14.3. - [Release notes](https://github.com/hyperium/tonic/releases) - [Changelog](https://github.com/hyperium/tonic/blob/master/CHANGELOG.md) - [Commits](hyperium/tonic@v0.14.2...v0.14.3) --- updated-dependencies: - dependency-name: tonic-prost-build dependency-version: 0.14.3 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [tonic-prost](https://github.com/hyperium/tonic) from 0.14.2 to 0.14.3. - [Release notes](https://github.com/hyperium/tonic/releases) - [Changelog](https://github.com/hyperium/tonic/blob/master/CHANGELOG.md) - [Commits](hyperium/tonic@v0.14.2...v0.14.3) --- updated-dependencies: - dependency-name: tonic-prost dependency-version: 0.14.3 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [tonic](https://github.com/hyperium/tonic) from 0.14.2 to 0.14.3. - [Release notes](https://github.com/hyperium/tonic/releases) - [Changelog](https://github.com/hyperium/tonic/blob/master/CHANGELOG.md) - [Commits](hyperium/tonic@v0.14.2...v0.14.3) --- updated-dependencies: - dependency-name: tonic dependency-version: 0.14.3 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
bbae391 to
40ac629
Compare
|
There was a strange merge conflict in Cargo.toml and I rebased the PR to |


Which issue does this PR close?
Part of #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_TUI_HTTP_TIMEOUT=3000Are there any user-facing changes?
New option for the CLI application,
--tuithat starts the TUI application