[
https://issues.apache.org/jira/browse/BOOKKEEPER-352?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13423896#comment-13423896
]
Hudson commented on BOOKKEEPER-352:
-----------------------------------
Integrated in bookkeeper-trunk #622 (See
[https://builds.apache.org/job/bookkeeper-trunk/622/])
BOOKKEEPER-352: Should not use static ServerStats/BKStats instance in
TestServerStats/TestBKStats (sijie via fpj) (Revision 1366384)
Result = SUCCESS
fpj :
Files :
* /zookeeper/bookkeeper/trunk/CHANGES.txt
*
/zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/proto/TestBKStats.java
*
/zookeeper/bookkeeper/trunk/hedwig-server/src/test/java/org/apache/hedwig/server/netty/TestServerStats.java
> Should not use static ServerStats/BKStats instance in
> TestServerStats/TestBKStats
> ---------------------------------------------------------------------------------
>
> Key: BOOKKEEPER-352
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-352
> Project: Bookkeeper
> Issue Type: Bug
> Affects Versions: 4.2.0
> Reporter: Sijie Guo
> Assignee: Sijie Guo
> Fix For: 4.2.0
>
> Attachments: BOOKKEEPER-352.diff
>
>
> TestServers failed when running whole test. Because TestServers uses
> ServerStats instance directly. ServerStats is a static instance, which will
> be used across whole testing. so the numSuccessOps would not be zero, the
> assertion would fail.
> Same problem occurs in TestBKStats. TestBKStats doesn't fail because
> statistics is turned off by default. The value happened to be zero. But it
> still is not a good idea to use static instance during testing.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
