-
Notifications
You must be signed in to change notification settings - Fork 428
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix async metrics test #3514
Fix async metrics test #3514
Conversation
Often flushes happen either because of a timeout, or because of explicit requests to flush the queue. Add an asynchronous wait for the metric to change, and add a metric increment of explicit flush requests.
Codecov Report
@@ Coverage Diff @@
## master #3514 +/- ##
==========================================
+ Coverage 81.02% 81.06% +0.04%
==========================================
Files 419 419
Lines 32309 32310 +1
==========================================
+ Hits 26177 26191 +14
+ Misses 6132 6119 -13
Continue to review full report at Codecov.
|
small_tests_24 / small_tests / 717ef07 small_tests_23 / small_tests / 717ef07 dynamic_domains_pgsql_mnesia_24 / pgsql_mnesia / 717ef07 dynamic_domains_mysql_redis_24 / mysql_redis / 717ef07 dynamic_domains_mssql_mnesia_24 / odbc_mssql_mnesia / 717ef07 dynamic_domains_pgsql_mnesia_23 / pgsql_mnesia / 717ef07 ldap_mnesia_23 / ldap_mnesia / 717ef07 ldap_mnesia_24 / ldap_mnesia / 717ef07 internal_mnesia_24 / internal_mnesia / 717ef07 pgsql_mnesia_24 / pgsql_mnesia / 717ef07 pgsql_mnesia_23 / pgsql_mnesia / 717ef07 elasticsearch_and_cassandra_24 / elasticsearch_and_cassandra_mnesia / 717ef07 mysql_redis_24 / mysql_redis / 717ef07 mssql_mnesia_24 / odbc_mssql_mnesia / 717ef07 riak_mnesia_24 / riak_mnesia / 717ef07 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks for the fix!
Often flushes happen either because of a timeout, or because of explicit
requests to flush the queue. Add an asynchronous wait for the metric to
change, and add a metric increment of explicit flush requests.