Re: Review Request: SQOOP-931 - Integration of Sqoop and HCatalog

2013-06-04 Thread Venkat Ranganathan
> On June 4, 2013, 11:15 p.m., Jarek Cecho wrote: > > Hi Venkat, > > Thank you for incorporating my comments, greatly appreciated. I've took a > > deep look again and I do have following additional comments: > > > > 1) Can we add the HCatalog tests into ThirdPartyTest suite? > > https://github

Re: Review Request: SQOOP-1028: remove poll-timeout parameter and add 'poll-timeout' option for sqoop

2013-06-04 Thread Jarek Cecho
> On June 5, 2013, 12:24 a.m., Jarek Cecho wrote: > > client/src/main/java/org/apache/sqoop/client/shell/SetOptionFunction.java, > > lines 76-77 > > > > > > What about throwing here exception about illegal input? > >

Re: Review Request: SQOOP-1028: remove poll-timeout parameter and add 'poll-timeout' option for sqoop

2013-06-04 Thread Mengwei Ding
> On June 5, 2013, 12:24 a.m., Jarek Cecho wrote: > > client/src/main/java/org/apache/sqoop/client/shell/SetOptionFunction.java, > > lines 76-77 > > > > > > What about throwing here exception about illegal input? I do

Re: Review Request: SQOOP-973 add --url option for 'set server', which enables specifying host, port and webapp in a more consistant way with other applications. For example, 'set server --url http://

2013-06-04 Thread Jarek Cecho
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11535/#review21450 --- Nice patch Mengwei! I do have one additional comment: client/src/m

Re: Review Request: SQOOP-1028: remove poll-timeout parameter and add 'poll-timeout' option for sqoop

2013-06-04 Thread Jarek Cecho
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11561/#review21449 --- client/src/main/java/org/apache/sqoop/client/shell/SetOptionFunctio

Re: Review Request: SQOOP-931 - Integration of Sqoop and HCatalog

2013-06-04 Thread Venkat Ranganathan
> On June 4, 2013, 11:15 p.m., Jarek Cecho wrote: > > Hi Venkat, > > Thank you for incorporating my comments, greatly appreciated. I've took a > > deep look again and I do have following additional comments: > > > > 1) Can we add the HCatalog tests into ThirdPartyTest suite? > > https://github

[jira] [Updated] (SQOOP-973) Sqoop2: Merge host, port and app to single server URL in Sqoop shell

2013-06-04 Thread Mengwei Ding (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-973?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mengwei Ding updated SQOOP-973: --- Attachment: SQOOP-973.1.patch > Sqoop2: Merge host, port and app to single server URL in Sqoop she

[jira] [Updated] (SQOOP-1028) Remove poll timeout parameter from Sqoop shell synchronous submission

2013-06-04 Thread Mengwei Ding (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-1028?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mengwei Ding updated SQOOP-1028: Attachment: SQOOP-1028.2.patch > Remove poll timeout parameter from Sqoop shell synchronous sub

[jira] [Updated] (SQOOP-973) Sqoop2: Merge host, port and app to single server URL in Sqoop shell

2013-06-04 Thread Mengwei Ding (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-973?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mengwei Ding updated SQOOP-973: --- Attachment: (was: SQOOP-973.1.batch) > Sqoop2: Merge host, port and app to single server URL i

Re: Review Request: SQOOP-1028: remove poll-timeout parameter and add 'poll-timeout' option for sqoop

2013-06-04 Thread Mengwei Ding
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11561/ --- (Updated June 4, 2013, 11:46 p.m.) Review request for Sqoop, Jarek Cecho, Hari

Re: Review Request: SQOOP-973 add --url option for 'set server', which enables specifying host, port and webapp in a more consistant way with other applications. For example, 'set server --url http://

2013-06-04 Thread Mengwei Ding
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11535/ --- (Updated June 4, 2013, 11:22 p.m.) Review request for Sqoop, Jarek Cecho, Hari

Re: Review Request: SQOOP-931 - Integration of Sqoop and HCatalog

2013-06-04 Thread Jarek Cecho
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10688/#review21420 --- Hi Venkat, Thank you for incorporating my comments, greatly apprecia

Re: Review Request: SQOOP-973 add --url option for 'set server', which enables specifying host, port and webapp in a more consistant way with other applications. For example, 'set server --url http://

2013-06-04 Thread Mengwei Ding
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11647/ --- (Updated June 4, 2013, 11:10 p.m.) Review request for Sqoop, Jarek Cecho, Hari

Review Request: SQOOP-973 add --url option for 'set server', which enables specifying host, port and webapp in a more consistant way with other applications. For example, 'set server --url http://loca

2013-06-04 Thread Mengwei Ding
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11647/ --- Review request for Sqoop, Jarek Cecho, Hari Shreedharan, and Abraham Elmahrek.

Review Request: SQOOP-973 add --url option for 'set server', which enables specifying host, port and webapp in a more consistant way with other applications. For example, 'set server --url http://loca

2013-06-04 Thread Mengwei Ding
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11645/ --- Review request for Sqoop, Jarek Cecho, Hari Shreedharan, and Abraham Elmahrek.

[jira] [Updated] (SQOOP-1063) Sqoop2: Export sub directories option

2013-06-04 Thread Vasanth kumar RJ (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-1063?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vasanth kumar RJ updated SQOOP-1063: Attachment: (was: SQOOP-1063.patch) > Sqoop2: Export sub directories option > -

[jira] [Updated] (SQOOP-1063) Sqoop2: Export sub directories option

2013-06-04 Thread Vasanth kumar RJ (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-1063?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vasanth kumar RJ updated SQOOP-1063: Attachment: SQOOP-1063.patch > Sqoop2: Export sub directories option >

Re: Review Request: Sqoop2: Export sub directories option

2013-06-04 Thread rj . vasanthkumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11588/ --- (Updated June 4, 2013, 11:58 a.m.) Review request for Sqoop. Changes ---