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/07/29 19:32:27 UTC

[GitHub] [orc] dongjoon-hyun commented on pull request #787: ORC-880: Reduce Visibility of DictionaryUtils

dongjoon-hyun commented on pull request #787:
URL: https://github.com/apache/orc/pull/787#issuecomment-889403816


   We cannot hide all of them because it's an unnecessarily breaking change. At 1.7.0, we already recovered many things to help the downstreams' migration. Although they are designed to be internal, someone use them. So, I'd rather close ORC-880 as 'Won't Do'.
   
   BTW, I really agree with your architectural design mind and your intention here. Sorry for not accepting your efforts, @belugabehr .


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