Major Description
We have code in SecureBulkLoadEndpoint#secureBulkLoadHFiles
LOG.trace("Setting permission for: " + p);
fs.setPermission(p, PERM_ALL_ACCESS);
This is against the point we use staging folder for secure bulk load. Currently we create a hidden staging folder which has ALL_ACCESS permission and we use "doAs" to move input files into staging folder. Therefore, we should not set 777 permission on the original input data files but files in staging folder after move.
This may comprise security setting especially when there is an error & we move the file with 777 permission back.