Github user hyunsik commented on the pull request:
https://github.com/apache/tajo/pull/423#issuecomment-83826070
Hi guys,
I agree with all you guys. This feature still requires unit tests, yet I
also don't want to double test time. We can compromise it.
First of all, this feature will completely work after we have a better
scheduler. So, we need to mark this feature as experimental for a while. As
navis said, we need to keep it disabled by default, and if necessary, users can
enable this feature with session variables.
With this approach, we may not need to verify all unit tests with this
feature right now. We won't double the unit test time. Nevertheless, since this
feature will require unit tests, it would be great if we test this this feature
in only some unit tests like TestJoinQuery and TestUnionQuery, which mostly
have independent execution blocks.
After we have a better scheduler, Tajo will use this feature by default. At
that time, all unit tests verify this feature.
---
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.
---
