Github user blrunner commented on the pull request:
https://github.com/apache/tajo/pull/459#issuecomment-93695955
Hi @navis
This looks good overall to me and I leaved some comments.
In addition, if possible, could rebase this again? I failed to merge as
follows.
```
Removing
tajo-rpc/tajo-rpc-protobuf/src/main/java/org/apache/tajo/rpc/RpcConnectionPool.java
Auto-merging
tajo-core/src/main/java/org/apache/tajo/master/QueryInProgress.java
Auto-merging
tajo-client/src/main/java/org/apache/tajo/client/SessionConnection.java
CONFLICT (content): Merge conflict in
tajo-client/src/main/java/org/apache/tajo/client/SessionConnection.java
Auto-merging
tajo-client/src/main/java/org/apache/tajo/client/QueryClientImpl.java
Automatic merge failed; fix conflicts and then commit the result.
```
---
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.
---
