Github user blrunner commented on the pull request:
https://github.com/apache/tajo/pull/476#issuecomment-87938171
It looks good Jihoon's opinion. And there are another way using
PreLogicalPlanVerifier or LogicalPlanVerifier.
But I would like you to proceed the umbrella issue in following order.
1. Implement ORCAppender and ORCScanner
2. Implement ORCStorageManager
3. Added ORC type to Tajo grammer.
I suggested above idea because we need to proceed development avoiding
users confusion. Also if you want the umbrella issue to merge to the apache
master branch, you need to complete all sub tasks before the next release. For
reference, I don't intend to push you and I just mean to help your schedule.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
