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/08/05 01:56:34 UTC

[GitHub] [orc] dongjoon-hyun edited a comment on pull request #810: ORC-904: Use Map for userMetadata in ReaderImpl

dongjoon-hyun edited a comment on pull request #810:
URL: https://github.com/apache/orc/pull/810#issuecomment-893102487


   I agree with you, @guiyanakuang .
   
   Although you revised your PR, I still suspect that some readers might expect the duplicated keys together (not the first one or the last one). Let's revisit this when the performance issue become severe.


-- 
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: dev-unsubscribe@orc.apache.org

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