Skip to content

Add 200000 microsecond sleep in E2E-PHP#4582

Merged
wu-sheng merged 15 commits intoapache:masterfrom
heyanlong:master
Mar 28, 2020
Merged

Add 200000 microsecond sleep in E2E-PHP#4582
wu-sheng merged 15 commits intoapache:masterfrom
heyanlong:master

Conversation

@heyanlong
Copy link
Member

@heyanlong heyanlong commented Mar 28, 2020

Please answer these questions before submitting pull request

  • Why submit this pull request?

  • Bug fix

  • New feature provided

  • Improve performance

  • Related issues


Bug fix

  • Bug description.

  • How to fix?


New feature or improvement

  • Describe the details and related test reports.

@wu-sheng wu-sheng added this to the 7.1.0 milestone Mar 28, 2020
@wu-sheng wu-sheng added the test Test requirements about performance, feature or before release. label Mar 28, 2020
@kezhenxu94
Copy link
Member

why sleep so long unnecessarily? This will slow down the checks drastically

@heyanlong
Copy link
Member Author

why sleep so long unnecessarily? This will slow down the checks drastically

simulation 200ms processing time

@wu-sheng wu-sheng merged commit 9c62e4f into apache:master Mar 28, 2020
@kezhenxu94
Copy link
Member

kezhenxu94 commented Mar 28, 2020

why sleep so long unnecessarily? This will slow down the checks drastically

simulation 200ms processing time

Your title is misleading, which says 200000ms=200s=~2minutes

@wu-sheng
Copy link
Member

@kezhenxu94 I think the title is microsecond, not ms, which usually is meaning millionseconds

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Test requirements about performance, feature or before release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants