Github user jihoonson commented on the pull request:
https://github.com/apache/tajo/pull/434#issuecomment-84746926
I think that it is possible to prune unnecessary partitions for the last
two cases without hurting our philosophy as follows.
When visiting a partition directory,
* if the current directory is leaf, read all files.
* otherwise,
* if some conditions are given for the current partition, visit only
qualified sub-directory.
* otherwise, visit all sub-directory.
Anyway, here is my +1. I'll commit it.
However, I missed one more comment. EvalTreeUtil is the more proper class
for checkIfPartitionSelection() and getPartitionValue() functions rather than
PlannerUtil because every function related to EvalNode is located in that class.
If you don't mind, I'll move those functions to EvalTreeUtil before commit.
---
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.
---
