| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Hi @timsaucer , if you could please take a pass on this PR , that would be great, thanks! |
Sorry, something went wrong.
|
This is great. We had someone open a related issue #1103 and it would be fantastic if we could address both of them at once. Do you think you could enhance this PR to handle python based catalogs and schemas as well? It is something I was planning to work on but I've been hard at work trying to get metadata handling in the upstream repo. I still have a bit of work there before I can focus on this repo again. Which is to say, I can take what you have here (which is great) and work from it if you need. Does that all make sense? |
Sorry, something went wrong.
yes, I can try to do so, I will go and read what this is about and come back :) |
Sorry, something went wrong.
|
Thank you for this excellent work and for the review of the other PR. Do you think we can close this one? |
Sorry, something went wrong.
happy to help :) if you have some other issues to be work on, please send them my way I will make my way through
yes, please, thanks @timsaucer ! |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Which issue does this PR close?
Closes #1091
Rationale for this change
Similar to exposing FFI for TableProviders, this PR exposes the capability for exposing CatalogProvider in python, now that apache/datafusion#15280 was merged.
What changes are included in this PR?
Includes the necessary changes to expose the FFICatalogProvider, an example, and the corresponding use case.
Are there any user-facing changes?
Yes, the new register_catalog method in the SessionContext
No breaking changes.