Skip to content
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

Update datafusion-cli to support Ballista, or implement new ballista-cli #886

Closed
andygrove opened this issue Aug 14, 2021 · 1 comment · Fixed by #889
Closed

Update datafusion-cli to support Ballista, or implement new ballista-cli #886

andygrove opened this issue Aug 14, 2021 · 1 comment · Fixed by #889
Assignees
Labels
enhancement New feature or request

Comments

@andygrove
Copy link
Member

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
The DataFusion CLI is very handy for demos and testing. It would be nice if we had a similar capability for Ballista.

Describe the solution you'd like
I would like a CLI that works with Ballista.

Describe alternatives you've considered
Add Ballista support to datafusion-cli, or create a new ballista-cli crate.

Additional context
N/A

@alamb
Copy link
Contributor

alamb commented Aug 15, 2021

I think a single CLI would be the best as it reduces the number of crates to keep track of (for users and contributors) as well as makes the close connection of ballista/datafusion apparent.

Thus I would favor extending datafusion-cli to support ballista, though either solution would be fine with me

If we are worried (as I am sometimes) about an unreasonable number of additional dependencies, we could add a ballista rust feature to the datafusion-cli to conditionally include ballista

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants