[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16277684#comment-16277684 ] Hudson commented on HBASE-19159: FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #4169 (See

[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16277344#comment-16277344 ] Vladimir Rodionov commented on HBASE-19159: --- lgtm, thanks for the patch [~gubjanos]. > Backup

[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16276605#comment-16276605 ] Hadoop QA commented on HBASE-19159: --- | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote ||

[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16272949#comment-16272949 ] Ted Yu commented on HBASE-19159: Please go thru checkstyle warnings. > Backup should check permission

[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16272939#comment-16272939 ] Ted Yu commented on HBASE-19159: Makes sense. In the next patch, can you add small test annotation to

[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16272895#comment-16272895 ] Hadoop QA commented on HBASE-19159: --- | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem

[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16265448#comment-16265448 ] Ted Yu commented on HBASE-19159: bq. maybe it should check WRITE_EXECUTE I searched thru hadoop tests -

[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16265381#comment-16265381 ] Ted Yu commented on HBASE-19159: Trying to see whether access() is documented in:

[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16265204#comment-16265204 ] Ted Yu commented on HBASE-19159: {code} 612 public static void checkBackupAccess(FileSystem fs,

[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16263819#comment-16263819 ] Ted Yu commented on HBASE-19159: I took a brief look at how hadoop tests similar scenario. Please refer

[jira] [Commented] (HBASE-19159) Backup should check permission for snapshot copy in advance

[ https://issues.apache.org/jira/browse/HBASE-19159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16257068#comment-16257068 ] Janos Gub commented on HBASE-19159: --- I would like to work on this issue. Attaching an initial patch,