> On March 12, 2016, 8:09 p.m., Venkat Ranganathan wrote: > > Thanks for working on this. Looking through this, I am trying to make > > sure we can build (but not have the functionality ) in older versions of > > hadoop and hbase. > > Is it possible to add reflection to identify and fail if the hadoop version > > and hbase version are not right. > > > > We always have newer capabilities from dependent products used (for example > > 2.6.0 Hadoop features are used for password alias), but we do reflection in > > those cases to not disable the building with older versions > > > > Venkat
Sounds a good idea Venkat. Will have it implemented and push another patch . - Ravi ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/#review123287 ----------------------------------------------------------- On March 11, 2016, 6:32 p.m., Ravi Magham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42809/ > ----------------------------------------------------------- > > (Updated March 11, 2016, 6:32 p.m.) > > > Review request for Sqoop and Venkat Ranganathan. > > > Repository: sqoop-trunk > > > Description > ------- > > Support for importing data onto Apache Phoenix tables > > > Diffs > ----- > > build.xml 97e5502 > ivy.xml d84b88f > ivy/ivysettings.xml 2920c89 > src/docs/user/import.txt fbb32f5 > src/docs/user/phoenix-args.txt PRE-CREATION > src/docs/user/phoenix.txt PRE-CREATION > src/docs/user/validation.txt 27a78e2 > src/java/org/apache/sqoop/SqoopOptions.java 17751c7 > src/java/org/apache/sqoop/manager/ConnManager.java 1811ce0 > src/java/org/apache/sqoop/manager/SqlManager.java 768507b > src/java/org/apache/sqoop/mapreduce/ImportJobBase.java 9b6e1a0 > src/java/org/apache/sqoop/mapreduce/PhoenixBulkImportJob.java PRE-CREATION > src/java/org/apache/sqoop/mapreduce/PhoenixBulkImportMapper.java > PRE-CREATION > src/java/org/apache/sqoop/mapreduce/PhoenixImportJob.java PRE-CREATION > src/java/org/apache/sqoop/mapreduce/PhoenixImportMapper.java PRE-CREATION > src/java/org/apache/sqoop/phoenix/PhoenixConstants.java PRE-CREATION > src/java/org/apache/sqoop/phoenix/PhoenixSqoopWritable.java PRE-CREATION > src/java/org/apache/sqoop/phoenix/PhoenixUtil.java PRE-CREATION > src/java/org/apache/sqoop/tool/BaseSqoopTool.java 50dd67d > src/java/org/apache/sqoop/tool/ImportTool.java ad1d48b > src/test/org/apache/sqoop/phoenix/PhoenixBaseTestCase.java PRE-CREATION > src/test/org/apache/sqoop/phoenix/PhoenixBasicImportTest.java PRE-CREATION > src/test/org/apache/sqoop/phoenix/PhoenixBulkImportTest.java PRE-CREATION > src/test/org/apache/sqoop/phoenix/PhoenixQueryImportTest.java PRE-CREATION > > Diff: https://reviews.apache.org/r/42809/diff/ > > > Testing > ------- > > > Thanks, > > Ravi Magham > >
