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

Mingliang Liu updated HADOOP-13966:
-----------------------------------
    Attachment: HADOOP-13966-HADOOP-13345.001.patch

The v1 patch address all Steve's comments except #3.

# I found the native libs are loaded as read-only, so I think sharing between 
parallel tests should be fine?
# In the maven pom.xml we have to check if how {{test.build.data}} is set and 
copy the native-libs accordingly. However, it can be unset, or overridden in 
the maven-surefire-plugin. This seems not very straightforward to check this in 
pom.xml

Existing code for copying native-libs:
{code:title=hadoop-aws/pom.xml}
          <execution>
            <id>copy</id>
            <phase>test-compile</phase>
            <goals>
              <goal>copy-dependencies</goal>
            </goals>
            <configuration>
              <includeScope>test</includeScope>
              <includeTypes>so,dll,dylib</includeTypes>
              <outputDirectory>target/native-libs</outputDirectory>
            </configuration>
          </execution>
{code}

I'll discuss with Steve offline and post comments here.

> Add ability to start DDB local server in every test
> ---------------------------------------------------
>
>                 Key: HADOOP-13966
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13966
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3, test
>    Affects Versions: HADOOP-13345
>            Reporter: Steve Loughran
>            Assignee: Mingliang Liu
>         Attachments: HADOOP-13966-HADOOP-13345.000.patch, 
> HADOOP-13966-HADOOP-13345.001.patch
>
>
> the local in memory DDB starts up in only 2+ seconds, so we have to reason to 
> not use it in all our integration tests, if we add a switch to do this



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to