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 2020/11/05 18:10:51 UTC

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

pgaref edited a comment on pull request #564:
URL: https://github.com/apache/orc/pull/564#issuecomment-722532262


   Hey @dongjoon-hyun -- sure  API backward compatibility is definitely something we should keep an eye on -- I was personally trying to upgrade to 1.6 on HIVErecently and bumped into too many API incompatibilities.
   However, we could also use schema/schemaEvolution and SARGS to derive columnNames ( so we could maintain a deprecated getColumnNames() method without keeping the extra field in the Reader.Options).
   Would something like the above make sense?


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