{"action":"created","issue":{"url":"https://api.github.com/repos/dotnet/coreclr/issues/17909","repository_url":"https://api.github.com/repos/dotnet/coreclr","labels_url":"https://api.github.com/repos/dotnet/coreclr/issues/17909/labels{/name}","comments_url":"https://api.github.com/repos/dotnet/coreclr/issues/17909/comments","events_url":"https://api.github.com/repos/dotnet/coreclr/issues/17909/events","html_url":"https://github.com/dotnet/coreclr/issues/17909","id":320880402,"node_id":"MDU6SXNzdWUzMjA4ODA0MDI=","number":17909,"title":"Remove ConcurrentStack from System.Private.CoreLib ","user":{"login":"maryamariyan","id":5897654,"node_id":"MDQ6VXNlcjU4OTc2NTQ=","avatar_url":"https://avatars2.githubusercontent.com/u/5897654?v=4","gravatar_id":"","url":"https://api.github.com/users/maryamariyan","html_url":"https://github.com/maryamariyan","followers_url":"https://api.github.com/users/maryamariyan/followers","following_url":"https://api.github.com/users/maryamariyan/following{/other_user}","gists_url":"https://api.github.com/users/maryamariyan/gists{/gist_id}","starred_url":"https://api.github.com/users/maryamariyan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maryamariyan/subscriptions","organizations_url":"https://api.github.com/users/maryamariyan/orgs","repos_url":"https://api.github.com/users/maryamariyan/repos","events_url":"https://api.github.com/users/maryamariyan/events{/privacy}","received_events_url":"https://api.github.com/users/maryamariyan/received_events","type":"User","site_admin":false},"labels":[{"id":517090700,"node_id":"MDU6TGFiZWw1MTcwOTA3MDA=","url":"https://api.github.com/repos/dotnet/coreclr/labels/area-System.Collections","name":"area-System.Collections","color":"d4c5f9","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/dotnet/coreclr/milestones/18","html_url":"https://github.com/dotnet/coreclr/milestone/18","labels_url":"https://api.github.com/repos/dotnet/coreclr/milestones/18/labels","id":3246413,"node_id":"MDk6TWlsZXN0b25lMzI0NjQxMw==","number":18,"title":"3.0","description":".NET Core 3.0, next major release after 2.1. See [official release dates](https://github.com/dotnet/core/blob/master/roadmap.md) for details.\r\n\r\nWe will move important issues from Future to 3.0 as the priorities and dates for next release become more clear.","creator":{"login":"karelz","id":5776068,"node_id":"MDQ6VXNlcjU3NzYwNjg=","avatar_url":"https://avatars2.githubusercontent.com/u/5776068?v=4","gravatar_id":"","url":"https://api.github.com/users/karelz","html_url":"https://github.com/karelz","followers_url":"https://api.github.com/users/karelz/followers","following_url":"https://api.github.com/users/karelz/following{/other_user}","gists_url":"https://api.github.com/users/karelz/gists{/gist_id}","starred_url":"https://api.github.com/users/karelz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karelz/subscriptions","organizations_url":"https://api.github.com/users/karelz/orgs","repos_url":"https://api.github.com/users/karelz/repos","events_url":"https://api.github.com/users/karelz/events{/privacy}","received_events_url":"https://api.github.com/users/karelz/received_events","type":"User","site_admin":false},"open_issues":94,"closed_issues":63,"state":"open","created_at":"2018-04-05T22:09:22Z","updated_at":"2018-06-04T22:42:04Z","due_on":null,"closed_at":null},"comments":6,"created_at":"2018-05-07T17:04:42Z","updated_at":"2018-06-05T23:56:00Z","closed_at":null,"author_association":"MEMBER","body":"This issue is opened to track removing ConcurrentStack from System.Private.CoreLib \r\n\r\nReferring to [comment](https://github.com/dotnet/coreclr/pull/17800#issuecomment-385092192) _From @stephentoub on Apr 27, 2018, 1:57 PM PDT_:\r\n\r\n> ConcurrentStack is to my knowledge used in only one place in corelib, by PinnableBufferCache, which is in turn only used by Overlapped as a way to cache OverlappedData instances, and I think we could come up with something better than ConcurrentStack for this use case, in particular since every Push ends up allocating a Node in ConcurrentStack.\r\n\r\nRelated to issue: https://github.com/dotnet/coreclr/issues/17751\r\n\r\ncc: @stephentoub @danmosemsft @jkotas "},"comment":{"url":"https://api.github.com/repos/dotnet/coreclr/issues/comments/394897017","html_url":"https://github.com/dotnet/coreclr/issues/17909#issuecomment-394897017","issue_url":"https://api.github.com/repos/dotnet/coreclr/issues/17909","id":394897017,"node_id":"MDEyOklzc3VlQ29tbWVudDM5NDg5NzAxNw==","user":{"login":"danmosemsft","id":6385855,"node_id":"MDQ6VXNlcjYzODU4NTU=","avatar_url":"https://avatars3.githubusercontent.com/u/6385855?v=4","gravatar_id":"","url":"https://api.github.com/users/danmosemsft","html_url":"https://github.com/danmosemsft","followers_url":"https://api.github.com/users/danmosemsft/followers","following_url":"https://api.github.com/users/danmosemsft/following{/other_user}","gists_url":"https://api.github.com/users/danmosemsft/gists{/gist_id}","starred_url":"https://api.github.com/users/danmosemsft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danmosemsft/subscriptions","organizations_url":"https://api.github.com/users/danmosemsft/orgs","repos_url":"https://api.github.com/users/danmosemsft/repos","events_url":"https://api.github.com/users/danmosemsft/events{/privacy}","received_events_url":"https://api.github.com/users/danmosemsft/received_events","type":"User","site_admin":false},"created_at":"2018-06-05T23:56:00Z","updated_at":"2018-06-05T23:56:00Z","author_association":"MEMBER","body":"Even if ConcurrentStack stays in CoreLib, why not remove it from CoreFX? We used to prefer to duplicate collections because the CoreFX one was easier to work on. But it can now easily be worked on in the mirrored Corelib within CoreFX.\r\n\r\nSame for ConcurrentQueue - CoreFX could consume from Corelib?"}}
{"actor":{"display_login":"danmosemsft"}}