upgle opened a new pull request, #5304:
URL: https://github.com/apache/openwhisk/pull/5304

   <!--- Provide a concise summary of your changes in the Title -->
   
   ## Description
   There is an issue that the test results are different depending on the 
performance of the build machine because the test code is written in a 
time-dependent. This PR improves memory queue test code that fails like 
heisenbug.
   
   - Use fake clock to improve time-dependent test code execution time.
     - The memory queue test execution time was reduced from 7m3s -> 1m34s in 
downstream environment.
   - Remove thread sleep and explicitly calls StateTimeout.
   
   ## Related issue and scope
   <!--- Please include a link to a related issue if there is one. -->
   - [ ] I opened an issue to propose and discuss this change (#????)
   
   ## My changes affect the following components
   <!--- Select below all system components are affected by your change. -->
   <!--- Enter an `x` in all applicable boxes. -->
   - [x] Scheduler
   - [x] Tests
   
   ## Types of changes
   - [x] Enhancement or new feature (adds new functionality).
   
   ## Checklist:
   <!--- Please review the points below which help you make sure you've covered 
all aspects of the change you're making. -->
   
   - [ ] I signed an [Apache 
CLA](https://github.com/apache/openwhisk/blob/master/CONTRIBUTING.md).
   - [ ] I reviewed the [style 
guides](https://github.com/apache/openwhisk/blob/master/CONTRIBUTING.md#coding-standards)
 and followed the recommendations (Travis CI will check :).
   - [ ] I added tests to cover my changes.
   - [ ] My changes require further changes to the documentation.
   - [ ] I updated the documentation where necessary.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to