----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/#review123287 -----------------------------------------------------------
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 - Venkat Ranganathan On March 11, 2016, 10:32 a.m., Ravi Magham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42809/ > ----------------------------------------------------------- > > (Updated March 11, 2016, 10:32 a.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 > >
