[jira] [Commented] (AMBARI-2156) zookeeper service check fails when zk client is not co-hosted with zk server.

["Yusaku Sako (JIRA)" <ji...@apache.org>]

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

Yusaku Sako commented on AMBARI-2156:
-------------------------------------

Just a minor nit,  but I think it'll be a bit simpler to do:
{code}
+ 'serviceName': this.get('content.serviceName')
+ 'actionName': this.get('content.serviceName') === 'ZOOKEEPER' ? 'ZOOKEEPER_QUORUM_SERVICE_CHECK' : this.get('content.serviceName') + '_SERVICE_CHECK'
{code}

The service name identifier is always uppercase by convention, and QUORUM only applies to ZOOKEEPER anyway.
                
> zookeeper service check fails when zk client is not co-hosted with zk server.
> -----------------------------------------------------------------------------
>
>                 Key: AMBARI-2156
>                 URL: https://issues.apache.org/jira/browse/AMBARI-2156
>             Project: Ambari
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 1.2.3
>            Reporter: Jaimin D Jetly
>            Assignee: Jaimin D Jetly
>              Labels: client
>             Fix For: 1.2.4
>
>         Attachments: AMBARI-2156_1.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira