charlesbluca commented on code in PR #527:
URL: 
https://github.com/apache/arrow-datafusion-python/pull/527#discussion_r1373298541


##########
conda/recipes/meta.yaml:
##########
@@ -41,7 +41,6 @@ requirements:
     - {{ compiler('rust') }}
     - cargo-bundle-licenses
     - maturin >=0.15,<0.16
-    - libprotobuf =3

Review Comment:
   Maybe it would make sense to split this PR out into changes affecting the 
Python and Rust package outputs? From dask-sql's perspective we're really only 
worried about the crate, where it's more straightforward to opt in or out of 
features, but the Python package landscape is a little different and I'm not 
really familiar with the consumers of the conda-forge/PyPI package to know if 
they're using the Substrait components



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to