Re: [VOTE] Release Sqoop version 1.4.7

2018-01-09 Thread Venkat Ranganathan
+1 Venkat On 1/9/18, 10:25 AM, "Kathleen Ting" wrote: Thanks Attila for being the release manager. +1 on Sqoop 1.4.7 RC0. Thanks, Kate On Fri, Jan 5, 2018 at 3:17 AM, Boglarka Egyed wrote: > Hi Attila, > > Thank you for preparing this release

Re: [VOTE] Release Sqoop version 1.4.7

2018-01-09 Thread Kathleen Ting
Thanks Attila for being the release manager. +1 on Sqoop 1.4.7 RC0. Thanks, Kate On Fri, Jan 5, 2018 at 3:17 AM, Boglarka Egyed wrote: > Hi Attila, > > Thank you for preparing this release! > > I have checked the followings on RC0: > - checksums OK > - signature OK > - LICENSE and NOTICE files

Re: Removing com.cloudera.sqoop packages

2018-01-09 Thread Szabolcs Vasas
Hi Attila, Thank you for reviewing this patch so quickly! I have talked to Anna, she is going to review it very soon and post on ReviewBoard. Yes, I wanted to push this as one commit to trunk using the patch file, the small commits were only for reviewing. Szabolcs On Tue, Jan 9, 2018 at 3:46 P

Re: Removing com.cloudera.sqoop packages

2018-01-09 Thread Attila Szabó
Hey Szabi, Souds great! Two comments: AFAIR Anna Szonyi was also planning to do something similar last January (when she was busy around the build system, tests, gradle, etc.). It would make sense on my side, to reach out to her, maybe she's got some useful feedbacks for you too. I've opened an i

[jira] [Updated] (SQOOP-3273) Removing com.cloudera.sqoop packages

2018-01-09 Thread Szabolcs Vasas (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-3273?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Szabolcs Vasas updated SQOOP-3273: -- Attachment: SQOOP-3273.patch > Removing com.cloudera.sqoop packages > --

Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-09 Thread Szabolcs Vasas
> On Jan. 9, 2018, 1:46 p.m., Attila Szabo wrote: > > IMHO build.xml line 1098 (after this patch applied) should be also removed > > (this is an entry for the src/com/cloudera/sqoop directory to be included > > in the javadoc generation). > > > > Otherwise the patch looks okay, but please let

Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-09 Thread Szabolcs Vasas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65017/ --- (Updated Jan. 9, 2018, 2:25 p.m.) Review request for Sqoop. Changes ---

PLEASE FILL OUT: SQOOP USER SURVEY

2018-01-09 Thread Boglarka Egyed
Dear Sqoop Community, The Apache Sqoop Committee is interested in your input to help plan future versions, limit deprecation of popular features and help with community meetups. We would really appreciate you completing this short survey: https://progress.co1.qualtrics.com/jfe/form/SV_8BOsTjqhM2Jr

Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-09 Thread Attila Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65017/#review195030 --- IMHO build.xml line 1098 (after this patch applied) should be also

[jira] [Updated] (SQOOP-3273) Removing com.cloudera.sqoop packages

2018-01-09 Thread Szabolcs Vasas (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-3273?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Szabolcs Vasas updated SQOOP-3273: -- Attachment: SQOOP-3273.patch > Removing com.cloudera.sqoop packages > --

Re: Review Request 65017: Removing com.cloudera.sqoop packages

2018-01-09 Thread Szabolcs Vasas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65017/ --- (Updated Jan. 9, 2018, 12:52 p.m.) Review request for Sqoop. Changes ---

Re: Removing com.cloudera.sqoop packages

2018-01-09 Thread Ferenc Szabo
Hi Szabi, I believe this is a great idea. By removing these packages we will get rid of a great deal of technical debt that will simplify future change. I will also help to avoid unnecessary conversions like the ones I had to use in my recent SqoopOptions related change. So, also +1 from me! Ch