hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dinesh Chitlangia (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (HDDS-1918) hadoop-ozone-tools has integration tests run as unit
Date Tue, 06 Aug 2019 14:38:00 GMT

    [ https://issues.apache.org/jira/browse/HDDS-1918?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16901110#comment-16901110

Dinesh Chitlangia edited comment on HDDS-1918 at 8/6/19 2:37 PM:

[~adoroszlai] Thanks for filing the jira.

Option #1 sounds good to me.

was (Author: dineshchitlangia):
[~adoroszlai] Thanks for filing the jira.

In my first glance, none of the three test classes you listed make use of MiniOzoneCluster.

> hadoop-ozone-tools has integration tests run as unit
> ----------------------------------------------------
>                 Key: HDDS-1918
>                 URL: https://issues.apache.org/jira/browse/HDDS-1918
>             Project: Hadoop Distributed Data Store
>          Issue Type: Improvement
>          Components: build, test
>    Affects Versions: 0.4.1
>            Reporter: Doroszlai, Attila
>            Assignee: Doroszlai, Attila
>            Priority: Minor
> HDDS-1735 created separate test runner scripts for unit and integration tests.
> Problem: {{hadoop-ozone-tools}} tests are currently run as part of the unit tests, but
most of them start a {{MiniOzoneCluster}}, which is defined in {{hadoop-ozone-integration-test}}.
 Thus I think these tests are really integration tests, and should be run by {{integration.sh}}
instead.  There are currently only 3 real unit tests:
> {noformat}
> hadoop-ozone/tools/src/test/java/org/apache/hadoop/ozone/audit/parser/TestAuditParser.java
> hadoop-ozone/tools/src/test/java/org/apache/hadoop/ozone/freon/TestProgressBar.java
> hadoop-ozone/tools/src/test/java/org/apache/hadoop/ozone/genconf/TestGenerateOzoneRequiredConfigurations.java
> {noformat}
> {{hadoop-ozone-tools}} tests take ~6 minutes.
> Possible solutions in order of increasing complexity:
> # Run {{hadoop-ozone-tools}} tests in {{integration.sh}} instead of {{unit.sh}} (This
is similar to {{hadoop-ozone-filesystem}}, which is already run by {{integration.sh}} and
has 2 real unit tests.)
> # Move all integration test classes to the {{hadoop-ozone-integration-test}} module,
and make it depend on {{hadoop-ozone-tools}} and {{hadoop-ozone-filesystem}} instead of the
other way around.
> # Rename integration test classes to {{\*IT.java}} or {{IT\*.java}}, add filters for
Surefire runs.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: hdfs-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-help@hadoop.apache.org

View raw message