sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Venkat Ranganathan <n....@live.com>
Subject Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables
Date Sat, 12 Mar 2016 20:09:01 GMT

-----------------------------------------------------------
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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message