You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Cooper Gillan (Jira)" <ji...@apache.org> on 2020/02/17 04:49:00 UTC

[jira] [Commented] (AIRFLOW-6721) Reorganize and clean up Hive tests

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

Cooper Gillan commented on AIRFLOW-6721:
----------------------------------------

[~kamilbregula] - left a note in the old PR, but curious if you think this is on the right track:

https://github.com/apache/airflow/compare/master...coopergillan:AIRFLOW-6721-organize-hive-tests?expand=1

Would love any feedback on this. Thanks!

> Reorganize and clean up Hive tests
> ----------------------------------
>
>                 Key: AIRFLOW-6721
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-6721
>             Project: Apache Airflow
>          Issue Type: Sub-task
>          Components: tests
>    Affects Versions: 1.10.7
>            Reporter: Cooper Gillan
>            Assignee: Cooper Gillan
>            Priority: Major
>
> From [this PR comment|https://github.com/apache/airflow/pull/7316#issuecomment-580946162]:
> https://github.com/apache/airflow/blob/master/tests/providers/apache/hive/operators/test_hive.py#L29-L40
> This file contains tests for many unrelated components. I think it would be useful to transfer some tests to other files.
> Here is an example: PolideaInternal@a91b612
> To avoid conflicts you can do it in one PR, but it would be useful for each move to be in a separate commit. This will facilitate the review.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)