jujoramos commented on pull request #5175:
URL: https://github.com/apache/geode/pull/5175#issuecomment-670655764


   @mkevo 
   Thanks for adding the extra tests. I'm not fully convinced about the 
implemented fix, however, did you consider my comments 
[here](https://github.com/apache/geode/pull/5175#issuecomment-638793177)?. 
Can't you just simply invoke the `DiskStoreImpl.close()` method instead of 
modifying the `DiskStoreCommandsUtils` class, as it involves way less 
modifications and we're sure it won't affect other areas of the `disk` 
management stuff?.
   


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to