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

[ENH] Making Fugue[dask] depend on Dask-sql #1111

Open
goodwanghan opened this issue Apr 5, 2023 · 1 comment
Open

[ENH] Making Fugue[dask] depend on Dask-sql #1111

goodwanghan opened this issue Apr 5, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@goodwanghan
Copy link
Contributor

goodwanghan commented Apr 5, 2023

Is your feature request related to a problem? Please describe.
Currently dask-sql has integration with Fugue. The problem is some Fugue change will break this integration. Also Fugue-dask by default relies on the qpd project to support SQL, which I want to totally deprecate.

Describe the solution you'd like
Remove the dependency of fugue from this repo
Add the dependency of dask-sql on Fugue side
fugue-project/fugue#454

It's a good time to make the change now because you no longer depend on conda and Calcite.

Can you please wait a little bit for the related code change (I will make a PR to remove Fugue) before your next release?

@goodwanghan goodwanghan added enhancement New feature or request needs triage Awaiting triage by a dask-sql maintainer labels Apr 5, 2023
@ayushdg
Copy link
Collaborator

ayushdg commented Apr 5, 2023

Can you please wait a little bit for the related code change (I will make a PR to remove Fugue) before your next release?

We were planning on cutting a release EOD today/early tomm, to bump our dask version dependency to be compatible with the upcoming Rapids release.
That being said we're happy to do another patch release in the coming week or two and we can add the relevant code changes in then.

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

No branches or pull requests

2 participants