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

Todd Lipcon reassigned IMPALA-6335:
-----------------------------------

    Assignee: Todd Lipcon

> Remove the unnecessary decorator "pytest.mark.execute_serially" from tests 
> which can be run in parallel
> -------------------------------------------------------------------------------------------------------
>
>                 Key: IMPALA-6335
>                 URL: https://issues.apache.org/jira/browse/IMPALA-6335
>             Project: IMPALA
>          Issue Type: Improvement
>          Components: Infrastructure
>    Affects Versions: Impala 2.10.0
>            Reporter: Jinchul Kim
>            Assignee: Todd Lipcon
>            Priority: Minor
>              Labels: newbie
>
> Regarding the decorator "pytest.mark.execute_serially", I saw all the test 
> cases are marked as execute_serially in 
> tests/shell/test_shell_interactive.py. I guess a few tests should be run 
> exclusively, but the other tests do not require the serial option. What do 
> you think about this? I think we should consider to use the decorator when 
> adding test cases. Minimized serial run can help to reduce overall test 
> running time by parallelism.
> I think the following cases do not require the decorator.
> a. Check consistency from test result
> b. Test query cancellation
> c. Test shell options which are effective on local shell
> > From Tim's comment:
> I think you're right that many of the shell tests don't inherently require to 
> be executed serially. Some of them would require work to execute in parallel, 
> particularly the ones that inspect files like .impalahistory and tests that 
> check the values of global impala daemon metrics.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to