virajjasani commented on issue #348: HBASE-22643 : Delete region without 
archiving only if regiondir is pr…
URL: https://github.com/apache/hbase/pull/348#issuecomment-510170701
 
 
   > Hi @virajjasani had just made some review comments, can you share your 
thoughts on those?
   
   Thanks @wchevreuil. I agree with your comments and no need to check for 
existence of source file. Rather the only check required is if method is 
returning false. Also, this is just to prevent NPE so tests should not be done 
for IOE. Will make the changes

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


With regards,
Apache Git Services

Reply via email to