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

maoling commented on ZOOKEEPER-3912:
------------------------------------

I tested the admin command(*wget http://localhost:8080/commands/monitor)* and 
*echo mntr | nc 127.0.0.1 2183*, both showed *synced_followers* is float.
{code:java}
"synced_followers" : 2.0,  "revalidate_count" : 0.0,  "watch_bytes" : 0.0,
zk_synced_followers 2.0  zk_revalidate_count 0.0  zk_watch_bytes 0.0
{code}
But the code said *synced_followers* is a Integer.
{code:java}
*   - "fsync_threshold_exceed_count": Long
     *   - "followers": Integer (leader only)
     *   - "synced_followers": Integer (leader only)
     *   - "pending_syncs": Integer (leader only)
     */
    public static class MonitorCommand extends CommandBase
{code}
 

[~der_eismann]  Could you please help us improve the *mntr* related doc in the 
*zookeeperAdmin.md*

> 4LW MNTR command outputs in float when prometheus activated
> -----------------------------------------------------------
>
>                 Key: ZOOKEEPER-3912
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3912
>             Project: ZooKeeper
>          Issue Type: Bug
>          Components: metric system
>    Affects Versions: 3.6.1
>            Reporter: Philipp Trulson
>            Priority: Major
>
> We use zookeeper 3.6.1 in combination with solr 8.6.0 on our kubernetes 
> cluster. For monitoring purposes we activated the prometheus metrics for 
> zookeeper, which had the unexpected side effect that the 4lw mntr command now 
> also outputs floats instead of the usual integers. This broke the parser on 
> the solr side (SOLR-14752), because it was expecting ints.
> This ticket is to discuss if this is actually a bug or expected behavior 
> because this change isn't mentioned anywhere in the release notes or 
> documentation.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to