{"action":"created","issue":{"url":"https://api.github.com/repos/jgrapht/jgrapht/issues/123","labels_url":"https://api.github.com/repos/jgrapht/jgrapht/issues/123/labels{/name}","comments_url":"https://api.github.com/repos/jgrapht/jgrapht/issues/123/comments","events_url":"https://api.github.com/repos/jgrapht/jgrapht/issues/123/events","html_url":"https://github.com/jgrapht/jgrapht/pull/123","id":60916442,"number":123,"title":"Open up Specifics to allow for custom backing containers","user":{"login":"cwensel","id":24684,"avatar_url":"https://avatars.githubusercontent.com/u/24684?v=3","gravatar_id":"","url":"https://api.github.com/users/cwensel","html_url":"https://github.com/cwensel","followers_url":"https://api.github.com/users/cwensel/followers","following_url":"https://api.github.com/users/cwensel/following{/other_user}","gists_url":"https://api.github.com/users/cwensel/gists{/gist_id}","starred_url":"https://api.github.com/users/cwensel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cwensel/subscriptions","organizations_url":"https://api.github.com/users/cwensel/orgs","repos_url":"https://api.github.com/users/cwensel/repos","events_url":"https://api.github.com/users/cwensel/events{/privacy}","received_events_url":"https://api.github.com/users/cwensel/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-03-12T22:01:34Z","updated_at":"2015-03-15T03:28:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jgrapht/jgrapht/pulls/123","html_url":"https://github.com/jgrapht/jgrapht/pull/123","diff_url":"https://github.com/jgrapht/jgrapht/pull/123.diff","patch_url":"https://github.com/jgrapht/jgrapht/pull/123.patch"},"body":"For cases where a Graph may hold mutable objects (or objects out of the developers influence), backing the graph with an IdentityHashMap both removes the nagging ‘vertex not found’ exceptions, it offers opportunities for performance gains.\r\n\r\nI’ve opened Specifics and added two factory create methods on AbstractBaseGraph, and further improved visibility to specific members allowing sub-classes of AbstractBaseGraph to provide a custom Specifics sub-class, or simply pass a backing Map<> to the constructor.\r\n\r\nThere are also tests mirroring the SimpleDirectedGraphTest using a Identity backed map.\r\n\r\nHappy to make any further changes to meet the project standards. Does look like some whitespace stuff leaked in, happy to clean that up and update the pr."},"comment":{"url":"https://api.github.com/repos/jgrapht/jgrapht/issues/comments/80820546","html_url":"https://github.com/jgrapht/jgrapht/pull/123#issuecomment-80820546","issue_url":"https://api.github.com/repos/jgrapht/jgrapht/issues/123","id":80820546,"user":{"login":"cwensel","id":24684,"avatar_url":"https://avatars.githubusercontent.com/u/24684?v=3","gravatar_id":"","url":"https://api.github.com/users/cwensel","html_url":"https://github.com/cwensel","followers_url":"https://api.github.com/users/cwensel/followers","following_url":"https://api.github.com/users/cwensel/following{/other_user}","gists_url":"https://api.github.com/users/cwensel/gists{/gist_id}","starred_url":"https://api.github.com/users/cwensel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cwensel/subscriptions","organizations_url":"https://api.github.com/users/cwensel/orgs","repos_url":"https://api.github.com/users/cwensel/repos","events_url":"https://api.github.com/users/cwensel/events{/privacy}","received_events_url":"https://api.github.com/users/cwensel/received_events","type":"User","site_admin":false},"created_at":"2015-03-15T03:28:08Z","updated_at":"2015-03-15T03:28:08Z","body":"I pushed a new commit with the initialization moved to the default constructor."}}