timsaucer opened a new pull request, #1156:
URL: https://github.com/apache/datafusion-python/pull/1156
# Which issue does this PR close?
Closes #1091
# Rationale for this change
This PR builds on top of
https://github.com/apache/datafusion-python/pull/1137 and adds python based
schema and catalog providers.
# What changes are included in this PR?
Creates wrapper types around python based schema and catalog providers.
Adds checks for when we are going python->rust->python and short circuit to
return the python original object rather than wrapper.
Adds unit test
# Are there any user-facing changes?
No
# Status
Leaving as draft until the following are complete:
- [ ] We need abstract base classes for the catalog and schema providers for
users to implement
- [ ] Verify all combinations of default vs user provided python catalog,
schema, table providers work in unit tests - this is mostly done
- [ ] Add full end-to-end test of registering custom catalog, schema, table
and then do a SQl query against it
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
