You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2023/10/25 20:32:01 UTC

[PR] Minor: reduce use of `#cfg(feature = "parquet")` in tests [arrow-datafusion]

alamb opened a new pull request, #7930:
URL: https://github.com/apache/arrow-datafusion/pull/7930

   ## Which issue does this PR close?
   
   Follow on to https://github.com/apache/arrow-datafusion/pull/7745/files
   
   ## Rationale for this change
   I noticed a few simplifications on https://github.com/apache/arrow-datafusion/pull/7745 -- namely since almost all the tests in enforce_distribution use the parquet 
   
   ## What changes are included in this PR?
   
   1. Replace a bunch of`#cfg(feature = "parquet")` with a test on the entire module
   
   ## Are these changes tested?
   
   Existing tests
   
   ## Are there any user-facing changes?
   No


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

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


Re: [PR] Minor: reduce use of `#cfg(feature = "parquet")` in tests [arrow-datafusion]

Posted by "andygrove (via GitHub)" <gi...@apache.org>.
andygrove merged PR #7930:
URL: https://github.com/apache/arrow-datafusion/pull/7930


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

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