shangxinli edited a comment on pull request #928: URL: https://github.com/apache/parquet-mr/pull/928#issuecomment-933499949
@gszadovszky Just addressed your feedback. For the conflict, I tried to merge from upstream but it always resulted in a lot more conflict. So I just simply revert my change that partially renamed AAD to pageHeaderAAD. I looked at the util.java file and I see there are a lot of other places all using AAD instead of a more specific AAD. We can have another PR to fix them all. -- 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]
