You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "Stamatis Zampetakis (Jira)" <ji...@apache.org> on 2022/03/10 10:49:00 UTC

[jira] [Comment Edited] (HIVE-26021) Change integration tests under DBInstallBase to regular unit tests

    [ https://issues.apache.org/jira/browse/HIVE-26021?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17504179#comment-17504179 ] 

Stamatis Zampetakis edited comment on HIVE-26021 at 3/10/22, 10:48 AM:
-----------------------------------------------------------------------

[~pvary] actually we were already running the following in CI:
* ITestMysql
* ITestPostgres
* ITestDerby

and after merging HIVE-26022 we are also running ITestOracle.

The main inconvenience is that we need special arguments in order to run them locally and special instructions in the Jenkinsfile. 


was (Author: zabetak):
[~pvary] actually we were already running the following in CI:
* ITestMysql
* ITestPostgres
* ITestDerby
and after merging HIVE-26022 we are also running ITestOracle.

The main inconvenience is that we need special arguments in order to run them locally and special instructions in the Jenkinsfile. 

> Change integration tests under DBInstallBase to regular unit tests
> ------------------------------------------------------------------
>
>                 Key: HIVE-26021
>                 URL: https://issues.apache.org/jira/browse/HIVE-26021
>             Project: Hive
>          Issue Type: Improvement
>          Components: Tests
>            Reporter: Stamatis Zampetakis
>            Priority: Major
>
> After HIVE-18588, some tests including those under [DBInstallBase|https://github.com/apache/hive/blob/1139c4b14db82a9e2316196819b35cfb713f34b5/standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/dbinstall/DbInstallBase.java] class have been marked as integration tests mainly to keep the test duration low.
> Nowadays, Hive developers rarely run all tests locally so separating between integration tests and unit tests does not provide a clear benefit. The separation adds maintenance cost and makes their execution more difficult scaring people away.
> The goal of this issue is to change the tests under {{DBInstallBase}} from "integration" tests back to regular unit tests and run them as part of the standard maven test phase without any fancy arguments.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)