-
Notifications
You must be signed in to change notification settings - Fork 87
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
Add remaining non-wrapped functions #767
Comments
Question: Have you ever used or do you know of a tool to run queries over python / rust codebases? It would be nice if we could generate a concrete report of what is not exposed. |
No, but I did write a small script to check and this is what I see missing:
Code to generate:
I can work on adding these tomorrow morning and I can also add this code as a unit test. |
FWIW I don't know if all of these need to be exported. It's probably worth looking through each one. |
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
We still have a few classes that do not yet have wrapper functions. Namely
datafusion.object_store
anddatafusion.common
. Additionally indatafusion.substrait
we referenceLogicalPlan
that is not exposed.Also worth doing is reviewing the excellent PR #751 to see how it now fits in with the updated python wrappers.
Describe the solution you'd like
Add missing wrappers and validate namespace corrections
Describe alternatives you've considered
None
Additional context
This is follow on work to #750
The text was updated successfully, but these errors were encountered: