[ 
https://issues.apache.org/jira/browse/TAJO-1393?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14368758#comment-14368758
 ] 

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_r26739002
  
    --- Diff: tajo-cli/src/main/java/org/apache/tajo/cli/tools/TajoHAAdmin.java 
---
    @@ -131,6 +131,9 @@ public void runCommand(String[] args) throws Exception {
         } else if (hostName == null && port == null) {
           tajoClient = new TajoClientImpl(ServiceTrackerFactory.get(tajoConf));
         }
    +    if (!tajoClient.isConnected()) {
    --- End diff --
    
    I think that this class also does not require TajoClient connection. 
Removing TajoClient would be a better solution to fix 'unused field' warning.


> 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)

Reply via email to