pavibhai commented on pull request #668:
URL: https://github.com/apache/orc/pull/668#issuecomment-854091482
> This is now merged to main! Thanks @pavibhai for all the hard work!
Thanks @pgaref
--
This is an automated message from the Apache Git Service.
To respond to the message,
pavibhai commented on pull request #668:
URL: https://github.com/apache/orc/pull/668#issuecomment-854091482
> This is now merged to main! Thanks @pavibhai for all the hard work!
Thanks @pgaref
--
This is an automated message from the Apache Git Service.
To respond to the message,
pavibhai commented on pull request #668:
URL: https://github.com/apache/orc/pull/668#issuecomment-844305261
> Hey @pavibhai thanks updating this! Any remaining comments to address? Is
this ready for a final review?
@pgaref thanks for following up. I think most of the changes are comp
pavibhai commented on pull request #668:
URL: https://github.com/apache/orc/pull/668#issuecomment-816806691
> Left some preliminary comments but my general advice would be to have one
final split of this where we just introduce the enums and prepare the Readers
for the actual planning/read
pavibhai commented on pull request #668:
URL: https://github.com/apache/orc/pull/668#issuecomment-815941563
@pgaref Following up on this to bring to your attention. If you get sometime
please go through the same and provide your feedback.
--
This is an automated message from the Apache G
pavibhai commented on pull request #668:
URL: https://github.com/apache/orc/pull/668#issuecomment-806031518
@pgaref @omalley
I have not broken this PR down any further so as to keep a merge
functionally complete as compared to just structural changes of adding a read
level for example.
