{"action":"created","issue":{"url":"https://api.github.com/repos/lewissbaker/cppcoro/issues/79","repository_url":"https://api.github.com/repos/lewissbaker/cppcoro","labels_url":"https://api.github.com/repos/lewissbaker/cppcoro/issues/79/labels{/name}","comments_url":"https://api.github.com/repos/lewissbaker/cppcoro/issues/79/comments","events_url":"https://api.github.com/repos/lewissbaker/cppcoro/issues/79/events","html_url":"https://github.com/lewissbaker/cppcoro/issues/79","id":312394040,"node_id":"MDU6SXNzdWUzMTIzOTQwNDA=","number":79,"title":"Random test failure of schedule_on() operator tests under msvc x86 debug","user":{"login":"lewissbaker","id":211814,"node_id":"MDQ6VXNlcjIxMTgxNA==","avatar_url":"https://avatars1.githubusercontent.com/u/211814?v=4","gravatar_id":"","url":"https://api.github.com/users/lewissbaker","html_url":"https://github.com/lewissbaker","followers_url":"https://api.github.com/users/lewissbaker/followers","following_url":"https://api.github.com/users/lewissbaker/following{/other_user}","gists_url":"https://api.github.com/users/lewissbaker/gists{/gist_id}","starred_url":"https://api.github.com/users/lewissbaker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lewissbaker/subscriptions","organizations_url":"https://api.github.com/users/lewissbaker/orgs","repos_url":"https://api.github.com/users/lewissbaker/repos","events_url":"https://api.github.com/users/lewissbaker/events{/privacy}","received_events_url":"https://api.github.com/users/lewissbaker/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-04-09T05:01:13Z","updated_at":"2019-03-04T21:18:31Z","closed_at":null,"author_association":"OWNER","body":"There have been some random test-failures on the CI servers.\r\n\r\neg. This [test run](https://ci.appveyor.com/project/lewissbaker/cppcoro/build/1.0.140/job/xwimhya3f2jwp024) failed with:\r\n```\r\nc:\\projects\\cppcoro\\test\\scheduling_operator_tests.cpp(21)\r\nTEST SUITE: schedule/resume_on\r\nTEST CASE:  schedule_on task<> function\r\n\r\nc:\\projects\\cppcoro\\test\\scheduling_operator_tests.cpp(40) ERROR!\r\n  CHECK( std::this_thread::get_id() == ioThreadId )\r\nwith expansion:\r\n  CHECK( 3936 == 3800 )\r\n```\r\n\r\nThere are a couple of working theories here:\r\n- caused by a compiler bug - msvc x86 coroutines support is still a little unreliable\r\n- caused by a lack of thread-safety in doctest - long shot as only one thread at a time should be executing CHECK statements\r\n- caused by a bug in `schedule_on()` operator itself - the implementation of this is a fairly thin wrapper over `co_await scheduler.schedule()`"},"comment":{"url":"https://api.github.com/repos/lewissbaker/cppcoro/issues/comments/469425556","html_url":"https://github.com/lewissbaker/cppcoro/issues/79#issuecomment-469425556","issue_url":"https://api.github.com/repos/lewissbaker/cppcoro/issues/79","id":469425556,"node_id":"MDEyOklzc3VlQ29tbWVudDQ2OTQyNTU1Ng==","user":{"login":"lewissbaker","id":211814,"node_id":"MDQ6VXNlcjIxMTgxNA==","avatar_url":"https://avatars1.githubusercontent.com/u/211814?v=4","gravatar_id":"","url":"https://api.github.com/users/lewissbaker","html_url":"https://github.com/lewissbaker","followers_url":"https://api.github.com/users/lewissbaker/followers","following_url":"https://api.github.com/users/lewissbaker/following{/other_user}","gists_url":"https://api.github.com/users/lewissbaker/gists{/gist_id}","starred_url":"https://api.github.com/users/lewissbaker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lewissbaker/subscriptions","organizations_url":"https://api.github.com/users/lewissbaker/orgs","repos_url":"https://api.github.com/users/lewissbaker/repos","events_url":"https://api.github.com/users/lewissbaker/events{/privacy}","received_events_url":"https://api.github.com/users/lewissbaker/received_events","type":"User","site_admin":false},"created_at":"2019-03-04T21:18:31Z","updated_at":"2019-03-04T21:18:31Z","author_association":"OWNER","body":"Update: MSVC still does not support symmetric transfer (at least for 2017 Update 9). I am still yet to test against 2019 builds but I have not heard that thes\r\n\r\nNote that I have since merged some of the changes to `task` from the `tailcall` branch that allows it to use symmetric-transfer to avoid these problems under clang builds.\r\nSee PR #91."}}
{"actor":{"display_login":"lewissbaker"}}