Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-04-12 Thread Ravi Magham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/ --- (Updated April 12, 2016, 7:02 a.m.) Review request for Sqoop and Venkat

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-04-06 Thread Venkat Ranganathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/#review127484 --- Ship it! Fix the identified changes and ship it. There are

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-21 Thread James Taylor
I've never heard of anyone using 0.95. It's never been in any distro as far as I know. On Mon, Mar 21, 2016 at 9:44 PM, Ravi Magham wrote: > > > > On March 22, 2016, 4:08 a.m., Venkat Ranganathan wrote: > > > build.xml, line 897 > > > < >

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-21 Thread Ravi Magham
> On March 22, 2016, 4:08 a.m., Venkat Ranganathan wrote: > > build.xml, line 897 > > > > > > Why do we need this? This was added as the version of hadoop that is passed to the build script should be at a minimum

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-21 Thread Venkat Ranganathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/#review124725 --- build.xml (line 101)

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-21 Thread Ravi Magham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/ --- (Updated March 22, 2016, 3:08 a.m.) Review request for Sqoop and Venkat

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-20 Thread Ravi Magham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/ --- (Updated March 21, 2016, 1:21 a.m.) Review request for Sqoop and Venkat

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-12 Thread Ravi Magham
> 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

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-12 Thread Venkat Ranganathan
--- 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

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-11 Thread Ravi Magham
--- 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

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-08 Thread Venkat Ranganathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/#review122618 --- The patch does not apply - Can you please rebase and upload a

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-08 Thread Venkat Ranganathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/#review122617 --- There are still a few whitespace issues. Will fix and validate

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-03-05 Thread Ravi Magham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/ --- (Updated March 6, 2016, 5:43 a.m.) Review request for Sqoop and Venkat

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-02-19 Thread Venkat Ranganathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/#review120024 --- Thanks for doing this. If you look at the RB patch, there are

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-02-13 Thread Ravi Magham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/ --- (Updated Feb. 14, 2016, 12:11 a.m.) Review request for Sqoop and Venkat

Re: Review Request 42809: SQOOP-2649 : Support for importing data onto Apache Phoenix tables

2016-02-01 Thread Ravi Magham
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42809/ --- (Updated Feb. 1, 2016, 8:49 a.m.) Review request for Sqoop and Venkat