[ 
https://issues.apache.org/jira/browse/SQOOP-955?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ahmed El Baz updated SQOOP-955:
-------------------------------

    Attachment: SQOOP-955.trunk.1.patch

Attaching a patch to fix Unit Test failures on Windows. 

Change details:

Provide a Windows version for hive.cmd under testdata/hive/bin

Construct paths using File and URL classes rather than string formatting. This 
takes care of abstracting paths to be OS independent.

Ensure that classpath uses the URL separator '/' rather than the default File 
seprator '\' on Windows.

Invoke the correct script (hive.cmd vs hive) based on the host OS in HiveImport

Change TestClobRef and TestBlobRef to use "true" for recursive delete, 
otherwise the directory delete fails on Windows with a "Directory not empty" 
message.

Note:
This change requires a Hadoop version supporting Windows. One way is to have it 
installed in local cache, and let Sqoop use it as follows:

ant clean test -Dhadoopversion=100 -Dhadoop.version=1.1.0-SNAPSHOT 
-Dresolvers=internal

I will make sure this is added into the documentation as part of the 
Documentation Sub task SQOOP-960
                
> Fix Sqoop unit test failures on Windows
> ---------------------------------------
>
>                 Key: SQOOP-955
>                 URL: https://issues.apache.org/jira/browse/SQOOP-955
>             Project: Sqoop
>          Issue Type: Sub-task
>    Affects Versions: 1.4.3
>            Reporter: Ahmed El Baz
>             Fix For: 1.4.4
>
>         Attachments: SQOOP-955.trunk.1.patch
>
>
> Some unit tests are failing when run on Windows. This happens mainly for File 
> System difference like File path format, file extension differences (e.g. 
> scripts have extension .cmd on windows as opposed to .sh), .. etc

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to