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

Posted to dev@orc.apache.org by GitBox <gi...@apache.org> on 2021/01/25 12:19:15 UTC

[GitHub] [orc] pgaref commented on pull request #564: ORC-274: Remove columnNames field from Reader.Options

pgaref commented on pull request #564:
URL: https://github.com/apache/orc/pull/564#issuecomment-766777506


   > Sorry for the delay here, @pgaref . I switch to from `Draft` to `Normal`. If this PR passes all three project CIs (Hive/Iceberg/Spark), we are good to go~
   
   Thanks @dongjoon-hyun  for reviving this ! 
   Just updated the PR to address the getColumnNames incompatibility -- if we wanted to use schema instead of predicate leaves we would have to move the method to the ReaderImpl class thus kept it simple here.
   
   


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