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. One more nit; it seems like we've added the same dependency as a 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: us...@infra.apache.org
