[GitHub] [incubator-pegasus] cauchy1988 opened a new pull request #836: fix: 2 ut test

[GitBox <gi...@apache.org>]

cauchy1988 opened a new pull request #836:
URL: https://github.com/apache/incubator-pegasus/pull/836


   ### What problem does this PR solve? <!--add issue link with summary if exists-->
   fix two ut test;
   
   ### What is changed and how does it work?
   (1)hotkey_collector_test.cpp 
   if all the 'coarse_collector.capture_data' async logic not run, but "coarse_collector.analyse_data(result)" executed will cause unit test fail; this phenomenon has appeared many times in my laptop computer and build machine in my company;
   
   (2)test_split.cpp
   variable 'count' should be initialized, if not will cause unit test fail randomly 
   
   ### Checklist <!--REMOVE the items that are not applicable-->
   
   ##### Tests <!-- At least one of them must be included. -->
   Unit test
   
   ##### Code changes
   only unit test related code change
   
   ##### Side effects
   only have impact on unit test
   
   ##### Related changes
   no relate changes
   


-- 
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: dev-unsubscribe@pegasus.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org