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

Drop hdfs functions from Ibis top-level module #2505

Closed
datapythonista opened this issue Oct 30, 2020 · 1 comment · Fixed by #2545
Closed

Drop hdfs functions from Ibis top-level module #2505

datapythonista opened this issue Oct 30, 2020 · 1 comment · Fixed by #2545
Labels
compatibility Compatibility related issues or PRs impala The Apache Impala backend
Milestone

Comments

@datapythonista
Copy link
Contributor

In #2497, ibis.HDFS, ibis.hdfs_connect... have been moved to the Impala backend where they belong, and started to raise a FutureWarning if called directly from the top ibis module.

After releasing (maybe more than once, not sure what will make sense), we should drop them them complete from ibis and leave them only in ibis.impala.

@datapythonista datapythonista added hdfs impala The Apache Impala backend compatibility Compatibility related issues or PRs labels Oct 30, 2020
@datapythonista
Copy link
Contributor Author

@jreback ok to remove those in the next version? Since we're already showing a FutureWarning and we are planning to release a major version next (2.0), I think this is reasonable.

Removing them will make my life easier and the code cleaner with the entrypoints stuff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compatibility Compatibility related issues or PRs impala The Apache Impala backend
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants