timsaucer opened a new pull request, #750:
URL: https://github.com/apache/datafusion-python/pull/750
# Which issue does this PR close?
Closes #749.
# Rationale for this change
As described the the [issue linked
above](https://github.com/apache/datafusion-python/issues/749), some python
users find the current approach to exposing the `pyo3` generated functions and
classes to be "unpythonic" (their words). This PR attempts to make a first
class user interface by adding python documentation and type annotations in the
place most python users will look. It has the added benefits of a cleaner
`help(datafusion.functions)` and similar calls.
# What changes are included in this PR?
- Exposes some functions to python that were not previously available
- Exposes some classes to python that were not previously available
- Adds wrappers around classes
- Adds wrappers around all functions within `datafusion.functions`
- Adds documentation strings for all classes and functions.
# Are there any user-facing changes?
- Simplifies the module naming. Instead of `datafusion.functions.functions`
it is simply `datafusion.functions`. Similar changes occur for other modules.
-
DRAFT: There are a few TODO statements left in as markers for myself. I will
remove this comment when all are complete.
--
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
