You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/04/20 15:58:14 UTC

[GitHub] [arrow-datafusion] tustvold opened a new pull request, #2289: Remove Parquet Empty Projection Workaround

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

   # Which issue does this PR close?
   
   Part of #1999 
   
   # Rationale for this change
   
   #2000 added a workaround to special-case empty projections, this has since been fixed upstream with https://github.com/apache/arrow-rs/issues/1537 and so we can remove this code
   
   # What changes are included in this PR?
   
   Removes custom empty projection logic
   
   # Are there any user-facing changes?
   
   No
   
   FYI @pjmore 
   


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

[GitHub] [arrow-datafusion] tustvold merged pull request #2289: Remove Parquet Empty Projection Workaround

Posted by GitBox <gi...@apache.org>.

tustvold merged PR #2289:
URL: https://github.com/apache/arrow-datafusion/pull/2289


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

[GitHub] [arrow-datafusion] tustvold commented on pull request #2289: Remove Parquet Empty Projection Workaround

Posted by GitBox <gi...@apache.org>.

tustvold commented on PR #2289:
URL: https://github.com/apache/arrow-datafusion/pull/2289#issuecomment-1104373871

   The CI failure appears to be reproducible but isn't in a component I've modified... I am rather confused, will come back to tomorrow... If someone could provide some pointers that would be amazing :sweat_smile: 


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