[ 
https://issues.apache.org/jira/browse/HBASE-4169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13080472#comment-13080472
 ] 

Lohit Vijayarenu commented on HBASE-4169:
-----------------------------------------

@Stack Thanks for reviewing the patch. Yes, I made this separate class because 
FSUtils already has many other methods which works across non HDFS, so I moved 
recoverFileLease alone to different class. Are you suggesting I move 
FSImplementedUtils and FSHDFSUtils to {{org.apache.hadoop.hbase.fs.util}}
Yes I was trying to avoid mapfs://. We have maprfs jar which includes client. I 
would planning to drop MapRFS implementation of recoverFileLease which would do 
chmod and update hbase-site.xml when running under maprfs. Would this work? 

> FSUtils LeaseRecovery for non HDFS FileSystems.
> -----------------------------------------------
>
>                 Key: HBASE-4169
>                 URL: https://issues.apache.org/jira/browse/HBASE-4169
>             Project: HBase
>          Issue Type: Bug
>          Components: util
>    Affects Versions: 0.90.3, 0.90.4
>            Reporter: Lohit Vijayarenu
>         Attachments: HBASE-4169.1.patch
>
>
> FSUtils.recoverFileLease uses HDFS's recoverLease method to get lease before 
> splitting hlog file.
> This might not work for other filesystem implementations. 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to