[
https://issues.apache.org/jira/browse/TAJO-1393?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14368751#comment-14368751
]
ASF GitHub Bot commented on TAJO-1393:
--------------------------------------
Github user hyunsik commented on a diff in the pull request:
https://github.com/apache/tajo/pull/417#discussion_r26738649
--- Diff: tajo-cli/src/main/java/org/apache/tajo/cli/tools/TajoGetConf.java
---
@@ -119,6 +119,9 @@ public void runCommand(String[] args, boolean tsqlMode)
throws Exception {
} else if (hostName == null && port == null) {
tajoClient = new TajoClientImpl(ServiceTrackerFactory.get(tajoConf));
}
+ if (!tajoClient.isConnected()) {
--- End diff --
Actually, this class does not need TajoClient. I think that removing
tajoClient is a better solution. So, you don't need the below
``System.out.print`` statement.
> Resolve findbug warnings on Tajo Cli Module
> -------------------------------------------
>
> Key: TAJO-1393
> URL: https://issues.apache.org/jira/browse/TAJO-1393
> Project: Tajo
> Issue Type: Sub-task
> Components: cli
> Reporter: Hyunsik Choi
> Assignee: Dongjoon Hyun
> Fix For: 0.11.0
>
> Attachments: TAJO-1393.patch
>
>
> See the title.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
