> On Oct. 1, 2013, 7:49 p.m., Jarek Cecho wrote: > > src/java/org/apache/sqoop/mapreduce/HBaseBulkImportJob.java, lines 149-152 > > <https://reviews.apache.org/r/14240/diff/5/?file=359511#file359511line149> > > > > This seems to be very dangerous as at some point everyone can read the > > files, wouldn't be much secure to simply change the owner to the hbase user?
It would be more secure but it doesn't work. I just tried and I get: Caused by: org.apache.hadoop.ipc.RemoteException(org.apache.hadoop.security.AccessControlException): Non-super user cannot change owner The actual alternative would be to use the new approach enabled by https://issues.apache.org/jira/browse/HBASE-5498 and described in https://cwiki.apache.org/confluence/display/HCATALOG/HBase+Secure+Bulk+Load But sqoop can't assume this patch is available, right? - Alexandre ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14240/#review26576 ----------------------------------------------------------- On Oct. 1, 2013, 8:30 p.m., Alexandre Normand wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14240/ > ----------------------------------------------------------- > > (Updated Oct. 1, 2013, 8:30 p.m.) > > > Review request for Sqoop, Jarek Cecho, Jean-Marc Spaggiari, and vasanthkumar. > > > Repository: sqoop-trunk > > > Description > ------- > > This is the patch to address some of the comments on Zhancheng Deng's review > request (https://reviews.apache.org/r/13052/) which was basically the change > to add the --bulk-load-dir option to support the HBase doBulkLoad function. > > > Diffs > ----- > > src/java/org/apache/sqoop/SqoopOptions.java 01805f9 > src/java/org/apache/sqoop/hbase/HBasePutProcessor.java 9ceb5bd > src/java/org/apache/sqoop/hbase/ToStringPutTransformer.java 5ccf311 > src/java/org/apache/sqoop/manager/SqlManager.java 2a4992d > src/java/org/apache/sqoop/mapreduce/HBaseBulkImportJob.java PRE-CREATION > src/java/org/apache/sqoop/mapreduce/HBaseBulkImportMapper.java PRE-CREATION > src/java/org/apache/sqoop/tool/BaseSqoopTool.java ebb1857 > > Diff: https://reviews.apache.org/r/14240/diff/ > > > Testing > ------- > > > Thanks, > > Alexandre Normand > >
