guiyanakuang commented on PR #1244: URL: https://github.com/apache/orc/pull/1244#issuecomment-1241886249
https://github.com/apache/orc/blob/d085b647eec6b4e0ffc61ee0481049507b741746/java/core/src/java/org/apache/orc/impl/RunLengthIntegerReaderV2.java#L389-L394 So I think it would be better to port the above check code to the following. What do you think? @cxzl25 : ) https://github.com/apache/orc/blob/d085b647eec6b4e0ffc61ee0481049507b741746/java/core/src/java/org/apache/orc/impl/TreeReaderFactory.java#L1553-L1575 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
