[ https://issues.apache.org/jira/browse/HBASE-21356?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16659643#comment-16659643 ]
Umesh Agashe commented on HBASE-21356: -------------------------------------- +1, lgtm. A few comments left on RB. > bulkLoadHFile API should ensure that rs has the source hfile's write > permission > ------------------------------------------------------------------------------- > > Key: HBASE-21356 > URL: https://issues.apache.org/jira/browse/HBASE-21356 > Project: HBase > Issue Type: Bug > Reporter: Zheng Hu > Assignee: Zheng Hu > Priority: Major > Fix For: 3.0.0, 2.2.0, 2.0.3, 2.1.2 > > Attachments: HBASE-21356.v1.patch > > > If the rs bulk load a HFile but has no write permission of it, we can read & > compact the hfile, but after the compaction finished, the HFile willl be > moved to archive directory, the HFileCleaner won't has permission to delete, > then the HFile will always be keep in HDFS. > Need check the file's write permission when run bulkLoadHFile at server side, > if no write permission, then reject. -- This message was sent by Atlassian JIRA (v7.6.3#76005)