{"action":"created","issue":{"url":"https://api.github.com/repos/palantir/atlasdb/issues/2484","repository_url":"https://api.github.com/repos/palantir/atlasdb","labels_url":"https://api.github.com/repos/palantir/atlasdb/issues/2484/labels{/name}","comments_url":"https://api.github.com/repos/palantir/atlasdb/issues/2484/comments","events_url":"https://api.github.com/repos/palantir/atlasdb/issues/2484/events","html_url":"https://github.com/palantir/atlasdb/pull/2484","id":265070112,"number":2484,"title":"add default getRanges concurrency level","user":{"login":"jnelson15","id":4187816,"avatar_url":"https://avatars0.githubusercontent.com/u/4187816?v=4","gravatar_id":"","url":"https://api.github.com/users/jnelson15","html_url":"https://github.com/jnelson15","followers_url":"https://api.github.com/users/jnelson15/followers","following_url":"https://api.github.com/users/jnelson15/following{/other_user}","gists_url":"https://api.github.com/users/jnelson15/gists{/gist_id}","starred_url":"https://api.github.com/users/jnelson15/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jnelson15/subscriptions","organizations_url":"https://api.github.com/users/jnelson15/orgs","repos_url":"https://api.github.com/users/jnelson15/repos","events_url":"https://api.github.com/users/jnelson15/events{/privacy}","received_events_url":"https://api.github.com/users/jnelson15/received_events","type":"User","site_admin":false},"labels":[{"id":365548862,"url":"https://api.github.com/repos/palantir/atlasdb/labels/component:%20api","name":"component: api","color":"5319e7","default":false}],"state":"open","locked":false,"assignee":{"login":"gsheasby","id":15610492,"avatar_url":"https://avatars2.githubusercontent.com/u/15610492?v=4","gravatar_id":"","url":"https://api.github.com/users/gsheasby","html_url":"https://github.com/gsheasby","followers_url":"https://api.github.com/users/gsheasby/followers","following_url":"https://api.github.com/users/gsheasby/following{/other_user}","gists_url":"https://api.github.com/users/gsheasby/gists{/gist_id}","starred_url":"https://api.github.com/users/gsheasby/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gsheasby/subscriptions","organizations_url":"https://api.github.com/users/gsheasby/orgs","repos_url":"https://api.github.com/users/gsheasby/repos","events_url":"https://api.github.com/users/gsheasby/events{/privacy}","received_events_url":"https://api.github.com/users/gsheasby/received_events","type":"User","site_admin":false},"assignees":[{"login":"gsheasby","id":15610492,"avatar_url":"https://avatars2.githubusercontent.com/u/15610492?v=4","gravatar_id":"","url":"https://api.github.com/users/gsheasby","html_url":"https://github.com/gsheasby","followers_url":"https://api.github.com/users/gsheasby/followers","following_url":"https://api.github.com/users/gsheasby/following{/other_user}","gists_url":"https://api.github.com/users/gsheasby/gists{/gist_id}","starred_url":"https://api.github.com/users/gsheasby/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gsheasby/subscriptions","organizations_url":"https://api.github.com/users/gsheasby/orgs","repos_url":"https://api.github.com/users/gsheasby/repos","events_url":"https://api.github.com/users/gsheasby/events{/privacy}","received_events_url":"https://api.github.com/users/gsheasby/received_events","type":"User","site_admin":false}],"milestone":null,"comments":5,"created_at":"2017-10-12T20:11:44Z","updated_at":"2017-10-17T14:16:32Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/palantir/atlasdb/pulls/2484","html_url":"https://github.com/palantir/atlasdb/pull/2484","diff_url":"https://github.com/palantir/atlasdb/pull/2484.diff","patch_url":"https://github.com/palantir/atlasdb/pull/2484.patch"},"body":"Adds a configurable default concurrency level for the new getRanges implementation. Realized when I started wiring up various products that most are probably just going to hard-code or configure a single value like this so probably makes sense to just put in Atlas. I'm hoping to eventually figure out a way to add throttling to this as well, but couldn't find a clean way at the moment to do that so will follow-up with that later.\r\n\r\nLocation of the configs is admittedly a little odd -- thread pool size is in the KVS config and this new config is in Atlas config. However, former should be a function of connections which is KVS-specific so I think that makes sense. Let me know if you disagree.\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/palantir/atlasdb/2484)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/palantir/atlasdb/issues/comments/337246076","html_url":"https://github.com/palantir/atlasdb/pull/2484#issuecomment-337246076","issue_url":"https://api.github.com/repos/palantir/atlasdb/issues/2484","id":337246076,"user":{"login":"jnelson15","id":4187816,"avatar_url":"https://avatars0.githubusercontent.com/u/4187816?v=4","gravatar_id":"","url":"https://api.github.com/users/jnelson15","html_url":"https://github.com/jnelson15","followers_url":"https://api.github.com/users/jnelson15/followers","following_url":"https://api.github.com/users/jnelson15/following{/other_user}","gists_url":"https://api.github.com/users/jnelson15/gists{/gist_id}","starred_url":"https://api.github.com/users/jnelson15/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jnelson15/subscriptions","organizations_url":"https://api.github.com/users/jnelson15/orgs","repos_url":"https://api.github.com/users/jnelson15/repos","events_url":"https://api.github.com/users/jnelson15/events{/privacy}","received_events_url":"https://api.github.com/users/jnelson15/received_events","type":"User","site_admin":false},"created_at":"2017-10-17T14:16:32Z","updated_at":"2017-10-17T14:16:32Z","author_association":"MEMBER","body":"@gsheasby any tips on where the issue may be? Don't see the gradle task explicitly ignoring any packages."}}
{"actor":{"display_login":"jnelson15"}}