{"action":"created","issue":{"url":"https://api.github.com/repos/hazelcast/hazelcast/issues/5672","labels_url":"https://api.github.com/repos/hazelcast/hazelcast/issues/5672/labels{/name}","comments_url":"https://api.github.com/repos/hazelcast/hazelcast/issues/5672/comments","events_url":"https://api.github.com/repos/hazelcast/hazelcast/issues/5672/events","html_url":"https://github.com/hazelcast/hazelcast/pull/5672","id":93996822,"number":5672,"title":"Backport of Agrona collections","user":{"login":"mtopolnik","id":490953,"avatar_url":"https://avatars.githubusercontent.com/u/490953?v=3","gravatar_id":"","url":"https://api.github.com/users/mtopolnik","html_url":"https://github.com/mtopolnik","followers_url":"https://api.github.com/users/mtopolnik/followers","following_url":"https://api.github.com/users/mtopolnik/following{/other_user}","gists_url":"https://api.github.com/users/mtopolnik/gists{/gist_id}","starred_url":"https://api.github.com/users/mtopolnik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mtopolnik/subscriptions","organizations_url":"https://api.github.com/users/mtopolnik/orgs","repos_url":"https://api.github.com/users/mtopolnik/repos","events_url":"https://api.github.com/users/mtopolnik/events{/privacy}","received_events_url":"https://api.github.com/users/mtopolnik/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-07-09T09:08:40Z","updated_at":"2015-07-09T10:22:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hazelcast/hazelcast/pulls/5672","html_url":"https://github.com/hazelcast/hazelcast/pull/5672","diff_url":"https://github.com/hazelcast/hazelcast/pull/5672.diff","patch_url":"https://github.com/hazelcast/hazelcast/pull/5672.patch"},"body":"Contributes a subset of Agrona collections to Hazelcast codebase:\r\n\r\n1. `BiInt2ObjectMap`\r\n2. `Int2ObjectHashMap`\r\n3. `IntHashSet`\r\n4. `Long2LongHashMap`\r\n5. `Long2ObjectHashMap`\r\n6. `LongHashSet`\r\n\r\n`Int2ObjectHashMap` is already used by the `client` code and `Long2ObjectHashMap` will be used in a feature planned for 3.6. There are probably more target usages around the current codebase where their application would save memory and increase performance."},"comment":{"url":"https://api.github.com/repos/hazelcast/hazelcast/issues/comments/119900583","html_url":"https://github.com/hazelcast/hazelcast/pull/5672#issuecomment-119900583","issue_url":"https://api.github.com/repos/hazelcast/hazelcast/issues/5672","id":119900583,"user":{"login":"mtopolnik","id":490953,"avatar_url":"https://avatars.githubusercontent.com/u/490953?v=3","gravatar_id":"","url":"https://api.github.com/users/mtopolnik","html_url":"https://github.com/mtopolnik","followers_url":"https://api.github.com/users/mtopolnik/followers","following_url":"https://api.github.com/users/mtopolnik/following{/other_user}","gists_url":"https://api.github.com/users/mtopolnik/gists{/gist_id}","starred_url":"https://api.github.com/users/mtopolnik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mtopolnik/subscriptions","organizations_url":"https://api.github.com/users/mtopolnik/orgs","repos_url":"https://api.github.com/users/mtopolnik/repos","events_url":"https://api.github.com/users/mtopolnik/events{/privacy}","received_events_url":"https://api.github.com/users/mtopolnik/received_events","type":"User","site_admin":false},"created_at":"2015-07-09T10:22:57Z","updated_at":"2015-07-09T10:22:57Z","body":">We already have Int2ObjectHashMapTestin com.hazelcast.client.protocol we may remove one of them.\r\n\r\nThis is already done: https://github.com/hazelcast/hazelcast/pull/5672/files#diff-de46e701b778043002544ee5681ff89e\r\n\r\n>I think the agrona name in the package has no value as you can put this info into javadoc.\r\n\r\nAttribution wasn't the motivation anyway (that's covered in `NOTICE.TXT`), it's just a lack of a better idea on my part. How about `com.hazelcast.util.collection` and put `CacheConcurrentHashMap` and `ConcurrentReferenceHashMap` there?"}}