Github user jihoonson commented on the pull request:
https://github.com/apache/tajo/pull/331#issuecomment-69137847
@hyunsik thanks for your review. I also think that we should resolve this
issue as soon as possible.
All changes look good to me, too. The only thing I concern is, as commented
above, the frequent hangs during unit tests. Whenever a zookeeper client fails
to establish a connection, the test waits for 10 ~ 50 seconds. I think that it
will significantly bother us.
I cannot find this error in the current master branch. Thus, I suspect that
an existing, but hidden error is exposed due to the changed HBase-related
configurations. Maybe we can fix it in another issue.
What do you think about it?
---
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.
---
