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 2020/10/09 09:48:21 UTC

[GitHub] [arrow] jorisvandenbossche commented on pull request #8301: ARROW-10100: [C++][Python][Dataset] Add ParquetFileFragment::Subset method

jorisvandenbossche commented on pull request #8301:
URL: https://github.com/apache/arrow/pull/8301#issuecomment-706084228


   @bkietz so I was now thinking it is actually fine to return a fragment with 0 row groups (it can then be the responsibility of the user to check for this, if they want to filter out the empty fragments out of their list of fragments). But, the problem is that an empty row group vector currently already means "all row groups" ... 
   Would it be fine to change empty vector to nullptr to indicate "all row groups" ?


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