{"action":"created","issue":{"url":"https://api.github.com/repos/usnistgov/dastard/issues/69","repository_url":"https://api.github.com/repos/usnistgov/dastard","labels_url":"https://api.github.com/repos/usnistgov/dastard/issues/69/labels{/name}","comments_url":"https://api.github.com/repos/usnistgov/dastard/issues/69/comments","events_url":"https://api.github.com/repos/usnistgov/dastard/issues/69/events","html_url":"https://github.com/usnistgov/dastard/issues/69","id":340477466,"node_id":"MDU6SXNzdWUzNDA0Nzc0NjY=","number":69,"title":"We still have race conditions, now in the data publishers","user":{"login":"joefowler","id":1229009,"node_id":"MDQ6VXNlcjEyMjkwMDk=","avatar_url":"https://avatars2.githubusercontent.com/u/1229009?v=4","gravatar_id":"","url":"https://api.github.com/users/joefowler","html_url":"https://github.com/joefowler","followers_url":"https://api.github.com/users/joefowler/followers","following_url":"https://api.github.com/users/joefowler/following{/other_user}","gists_url":"https://api.github.com/users/joefowler/gists{/gist_id}","starred_url":"https://api.github.com/users/joefowler/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joefowler/subscriptions","organizations_url":"https://api.github.com/users/joefowler/orgs","repos_url":"https://api.github.com/users/joefowler/repos","events_url":"https://api.github.com/users/joefowler/events{/privacy}","received_events_url":"https://api.github.com/users/joefowler/received_events","type":"User","site_admin":false},"labels":[{"id":693531211,"node_id":"MDU6TGFiZWw2OTM1MzEyMTE=","url":"https://api.github.com/repos/usnistgov/dastard/labels/bug","name":"bug","color":"ee0701","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/usnistgov/dastard/milestones/1","html_url":"https://github.com/usnistgov/dastard/milestone/1","labels_url":"https://api.github.com/repos/usnistgov/dastard/milestones/1/labels","id":3457428,"node_id":"MDk6TWlsZXN0b25lMzQ1NzQyOA==","number":1,"title":"Dastard 0.1","description":"Basic Dastard that can be used to run a TES array and store useful data.","creator":{"login":"joefowler","id":1229009,"node_id":"MDQ6VXNlcjEyMjkwMDk=","avatar_url":"https://avatars2.githubusercontent.com/u/1229009?v=4","gravatar_id":"","url":"https://api.github.com/users/joefowler","html_url":"https://github.com/joefowler","followers_url":"https://api.github.com/users/joefowler/followers","following_url":"https://api.github.com/users/joefowler/following{/other_user}","gists_url":"https://api.github.com/users/joefowler/gists{/gist_id}","starred_url":"https://api.github.com/users/joefowler/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joefowler/subscriptions","organizations_url":"https://api.github.com/users/joefowler/orgs","repos_url":"https://api.github.com/users/joefowler/repos","events_url":"https://api.github.com/users/joefowler/events{/privacy}","received_events_url":"https://api.github.com/users/joefowler/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":22,"state":"open","created_at":"2018-06-27T20:51:56Z","updated_at":"2018-07-12T15:07:55Z","due_on":"2018-07-14T07:00:00Z","closed_at":null},"comments":1,"created_at":"2018-07-12T03:34:04Z","updated_at":"2018-07-12T15:11:31Z","closed_at":null,"author_association":"MEMBER","body":"I'll look into it, but I mean that you can run `go test -race` to see."},"comment":{"url":"https://api.github.com/repos/usnistgov/dastard/issues/comments/404546652","html_url":"https://github.com/usnistgov/dastard/issues/69#issuecomment-404546652","issue_url":"https://api.github.com/repos/usnistgov/dastard/issues/69","id":404546652,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNDU0NjY1Mg==","user":{"login":"ggggggggg","id":5192577,"node_id":"MDQ6VXNlcjUxOTI1Nzc=","avatar_url":"https://avatars2.githubusercontent.com/u/5192577?v=4","gravatar_id":"","url":"https://api.github.com/users/ggggggggg","html_url":"https://github.com/ggggggggg","followers_url":"https://api.github.com/users/ggggggggg/followers","following_url":"https://api.github.com/users/ggggggggg/following{/other_user}","gists_url":"https://api.github.com/users/ggggggggg/gists{/gist_id}","starred_url":"https://api.github.com/users/ggggggggg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ggggggggg/subscriptions","organizations_url":"https://api.github.com/users/ggggggggg/orgs","repos_url":"https://api.github.com/users/ggggggggg/repos","events_url":"https://api.github.com/users/ggggggggg/events{/privacy}","received_events_url":"https://api.github.com/users/ggggggggg/received_events","type":"User","site_admin":false},"created_at":"2018-07-12T15:11:31Z","updated_at":"2018-07-12T15:11:31Z","author_association":"COLLABORATOR","body":"I think 3 is the go way, from go/sync, emphasis mine. \"Package sync provides basic synchronization primitives such as mutual exclusion locks. Other than the Once and WaitGroup types, most are intended for use by low-level library routines. **Higher-level synchronization is better done via channels and communication.**\"\r\n\r\nI agree it sounds a bit like letting the \"best become the enemy of the good-enough\", maybe just try to do future work in that style? Thats how I did `publishSync` and it wasn't too bad."}}
{"actor":{"display_login":"ggggggggg"}}