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

Jerry He commented on HBASE-16257:
----------------------------------

Hi, Enis

Thanks for the comments. I will update the patch as commented.

bq. Are these changes related?
Not directly related.  The intention was to put the CORRUPT_DIR_NAME always 
under hbase.rootdir and get rid of the property "hbase.hfile.quarantine.dir".  
It is not referenced or documented anywhere else.
Ok to remove? I can put a deprecate warning in the log.

bq. Is this conf coming from the remote(sink) cluster for the HFileReplicator?
Yes

> Move staging dir to be under hbase root dir
> -------------------------------------------
>
>                 Key: HBASE-16257
>                 URL: https://issues.apache.org/jira/browse/HBASE-16257
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Jerry He
>            Assignee: Jerry He
>            Priority: Minor
>             Fix For: 2.0.0
>
>         Attachments: HBASE-16257-v1.patch, HBASE-16257-v2.patch
>
>
> The hbase.bulkload.staging.dir defaults to hbase.fs.tmp.dir which then 
> defaults to
> {code}
> public static final String DEFAULT_TEMPORARY_HDFS_DIRECTORY = "/user/"
>       + System.getProperty("user.name") + "/hbase-staging";
> {code}
> This default would have problem on local file system standalone case.
> We can move the staging dir to be under hbase.rootdir.  We are bringing 
> secure bulkload to the core. It makes sense to bring it under core control as 
> well, instead of an optional property.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to