rm serial mode in exclusive case #43073
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
你的PR提交成功，感谢你对开源项目的贡献! |
luotao1
approved these changes
May 30, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR types
Others
PR changes
Others
Describe
在制定单测并发策略的时候，发现有一些case即使设定了并发度，仍无法并发（多见于独占case）。在分析这些case时，发现它们均设定了
RUN_SERIVAL = 1表明其必须独占。而在真实的实验中，这些case并不是非要独占执行，以一定的并发度执行仍然可以执行成功。因此，此PR将
RUN_SERIVAL = 1去掉，收益为coverage的独占case由原先3000s左右可以降到1000s左右，同时全量case的执行时间由7500s降到5200s左右，如下图。此PR未改变原并发度，因此单测timeout的个数仍很多，retry时间也达到了1000s左右，下个PR会解决该问题。
同时，下个PR会增加
SERIVAL的approve规则。