kszucs commented on a change in pull request #1287:
URL: https://github.com/apache/arrow-datafusion/pull/1287#discussion_r748626829



##########
File path: docs/source/python/index.rst
##########
@@ -41,8 +41,9 @@ Simple usage:
    import datafusion
    import pyarrow
 
-   # an alias
+   # create aliases
    f = datafusion.functions
+   col = datafusion.column

Review comment:
       We can re-export the `col` and `lit` aliases from 
`datafusion/__init__.py` too.

##########
File path: python/datafusion/__init__.py
##########
@@ -36,6 +38,7 @@
     "ScalarUDF",
     "column",
     "literal",
+    "functions",

Review comment:
       This is how the compute functions are exposed in pyarrow as well:
   
   ```py
   >>> import pyarrow
   >>> pyarrow.compute.cast
   Traceback (most recent call last):
     File "<stdin>", line 1, in <module>
     File 
"/Users/kszucs/.conda/envs/ibis39/lib/python3.9/site-packages/pyarrow/__init__.py",
 line 266, in __getattr__
       raise AttributeError(
   AttributeError: module 'pyarrow' has no attribute 'compute'
   >>> import pyarrow.compute
   >>> pyarrow.compute.cast
   <function cast at 0x109349160>
   ```




-- 
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...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to