[GitHub] [arrow] vertexclique edited a comment on pull request #7873: ARROW-9608: [Rust] Leaner feature gating for arrow in parquet

[GitBox <gi...@apache.org>]

vertexclique edited a comment on pull request #7873:
URL: https://github.com/apache/arrow/pull/7873#issuecomment-668032973


   Parquet crate is using arrow's default features which are `default = ["flight", "prettyprint"]` but flight is not needed, nor prettyprint for parquet. these are not used at all. Moreover, since features are additive, we can't override arrow dependencies features in any project that uses parquet if we continue to use default features included.
   + seems like we've added the same dependency as dev dependency. Which is also not needed because it is in the default dependency list.
   
   > ... it seems inverted that arrow depends on flight. I wonder if we have the dependency the wrong way around?
   
   Sorry, might be I am quite a bit tired, but I didn't understand the inversion part.


----------------------------------------------------------------
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.

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