zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From anmolnar <...@git.apache.org>
Subject [GitHub] zookeeper issue #546: ZOOKEEPER-3066 Expose on JMX of Followers the id of th...
Date Fri, 22 Jun 2018 12:03:58 GMT
Github user anmolnar commented on the issue:

    https://github.com/apache/zookeeper/pull/546
  
    Thank you for the quick turnaround @eolivelli . Mockito unit test is cool, exactly what
I meant to.
    
    I think it's okay to validate the existence of JMX property in the existing test (similar
checks were already there), but would you please remove checks for the *value* of the property?
I really don't want the hammer test failing in case of an incorrect JMX value.


---

Mime
View raw message