Apache9 commented on issue #1335: HBASE-24000 Simplify CommonFSUtils after 
upgrading to hadoop 2.10.0
URL: https://github.com/apache/hbase/pull/1335#issuecomment-604193880
 
 
   The test is strange, we wrap a already created procedure store with mockito 
and then call the recoverLease method again. Not sure why but after calling the 
real method of the procStore, the current stream is still fine but after we 
returned from the method of the mocked instance, the current stream has already 
been closed and cause the test to fail.
   
   Since the class is only used for data migration and will be removed in 
4.0.0, do not want to spend much time on it. Just catch the FSError exception 
when closing...

----------------------------------------------------------------
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


With regards,
Apache Git Services

Reply via email to