Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Sqoop, mail # dev - Review Request: Request to review patch for SQOOP-954: Create Sqoop runtime scripts to run Sqoop on Windows


+
Ahmed El Baz 2013-03-20, 23:56
+
Venkat Ranganathan 2013-03-22, 06:11
+
Ahmed El Baz 2013-03-22, 18:55
+
Jarek Cecho 2013-03-28, 22:19
+
Jarek Cecho 2013-03-22, 02:51
+
Ahmed El Baz 2013-03-22, 18:55
+
Ahmed El Baz 2013-03-29, 03:01
Copy link to this message
-
Re: Review Request: Request to review patch for SQOOP-954: Create Sqoop runtime scripts to run Sqoop on Windows
Venkat Ranganathan 2013-03-29, 19:03

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10055/#review18523
-----------------------------------------------------------
Hi Ahmed

Thanks for the new patch.  It looks good.  I still have one issue and suggestion.  The powershell script to generate the jar file is very good!  You are generating a jar file everytime and the jar file is generated under SQOOP_HOME.   There may be installations for the SQOOP_HOME may not be writable by user.   Also, I think the main motivation is to overcome the environment strings limitation.   Since JDK 1.6, Java has the ability to provide an option to provide a shortcut for all jars in a file (This probably should be done for the Unix classpaths also).   Please see http://docs.oracle.com/javase/6/docs/technotes/tools/windows/classpath.html  

I am thinking whether this should be a simpler change to just add all jars in SQOOP_LIB.  We have to say %SQOOP_HOME%\lib\*.   Of course, this introduces dependency on 1.6+ versions of JDK, but given that 1.5 is EOLed this should be OK

Thanks

- Venkat Ranganathan
On March 29, 2013, 3:01 a.m., Ahmed El Baz wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10055/
> -----------------------------------------------------------
>
> (Updated March 29, 2013, 3:01 a.m.)
>
>
> Review request for Sqoop.
>
>
> Description
> -------
>
> A patch implementing the Windows version of Sqoop run scripts. The scripts follow the same logic as there .sh counterparts.
> One difference is to create a Jar which references all classpath elements in its Manifest, and provide that jar as the single jar needed for Sqoop. The reason here is that in some cases if the number of classpath elements is large, HADOOP_CLASSPATH gets very long which causes failures in Windows since there is a limit to command lines.
> As a workaround, I added a step to wrap all jars in the classpath in a single jar, and then use that generated jar (this is also done in hadoop for Windows to handle similar issues)
> I did this in a utility script "BuildJar" which can be used for other components as well.
> This change is specific to Windows scripts, Linux scripts are not affected.
>
>
> This addresses bug SQOOP-954.
>     https://issues.apache.org/jira/browse/SQOOP-954
>
>
> Diffs
> -----
>
>   bin/BuildJar.ps1 PRE-CREATION
>   bin/configure-sqoop.cmd PRE-CREATION
>   bin/sqoop.cmd PRE-CREATION
>   conf/sqoop-env-template.cmd PRE-CREATION
>
> Diff: https://reviews.apache.org/r/10055/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Ahmed El Baz
>
>

+
Ahmed El Baz 2013-03-29, 22:23
+
Venkat Ranganathan 2013-04-14, 04:58
+
Venkat Ranganathan 2013-04-15, 18:21
+
Ahmed El Baz 2013-04-22, 03:29
+
Ahmed El Baz 2013-04-22, 03:26
+
Venkat Ranganathan 2013-04-22, 07:06
+
Ahmed El Baz 2013-04-22, 17:05
+
Ahmed El Baz 2013-04-23, 04:13
+
Jarek Cecho 2013-04-23, 18:38
+
Ahmed El Baz 2013-04-23, 18:43
+
Venkat Ranganathan 2013-04-23, 04:26
+
Venkat Ranganathan 2013-04-22, 23:04
+
Venkat Ranganathan 2013-04-23, 04:12
+
Ahmed El Baz 2013-04-23, 04:04
+
Ahmed El Baz 2013-04-23, 04:15
+
Venkat Ranganathan 2013-04-22, 22:49
+
Venkat Ranganathan 2013-04-22, 17:37
+
Ahmed El Baz 2013-04-22, 17:51