runzhiwang opened a new pull request #84:
URL: https://github.com/apache/incubator-ratis/pull/84


   What's the problem ?
   In unit test testServerRestartOnException,  when call 
ServerImplUtils.newRaftServer, it did not set storage dir in RaftProperties. 
Then it use the default storage dir /tmp/raft-server, but when test finish, it 
did not delete the generated sub dir. So every time 
testServerRestartOnException, it will generate a sub dir in /tmp/raft-server as 
the image shows. But when  testServerRestartOnException run, it will call 
RaftServerProxy::initGroups, and create a RaftServerImpl for each sub dir in 
/tmp/raft-server, which leads to a big noise for the test.
   
   
![image](https://user-images.githubusercontent.com/51938049/80867499-f6305f80-8cc6-11ea-813a-5ff9f7eca623.png)
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to