GitHub PullRequestEvent at 2014-11-17 16:03:46
Repository: ibrowse
Actor: benjaminplee
Payload: {"action":"opened","number":122,"pull_request":{"url":"https://api.github.com/repos/cmullaparthi/ibrowse/pulls/122","id":24545379,"html_url":"https://github.com/cmullaparthi/ibrowse/pull/122","diff_url":"https://github.com/cmullaparthi/ibrowse/pull/122.diff","patch_url":"https://github.com/cmullaparthi/ibrowse/pull/122.patch","issue_url":"https://api.github.com/repos/cmullaparthi/ibrowse/issues/122","number":122,"state":"open","locked":false,"title":"Status pipeline algorithm changes?","user":{"login":"benjaminplee","id":17233,"avatar_url":"https://avatars.githubusercontent.com/u/17233?v=3","gravatar_id":"","url":"https://api.github.com/users/benjaminplee","html_url":"https://github.com/benjaminplee","followers_url":"https://api.github.com/users/benjaminplee/followers","following_url":"https://api.github.com/users/benjaminplee/following{/other_user}","gists_url":"https://api.github.com/users/benjaminplee/gists{/gist_id}","starred_url":"https://api.github.com/users/benjaminplee/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benjaminplee/subscriptions","organizations_url":"https://api.github.com/users/benjaminplee/orgs","repos_url":"https://api.github.com/users/benjaminplee/repos","events_url":"https://api.github.com/users/benjaminplee/events{/privacy}","received_events_url":"https://api.github.com/users/benjaminplee/received_events","type":"User","site_admin":false},"body":"My team has been using iBrowse for some time now in one of our products without issue, thank you, but some recent scale testing found that the current find_best_connection logic was problematic with our internal load balancers between systems.  After reviewing the current code we have fielded (slightly behind current master e18ea912b62b0f4a48d0f0edc8bb2f542863a88c), I see our problem is with consistent attempts at giving the \"first\" connection the next request, even if others have fewer outstanding in their pipeline or round-robin.  I was about to begin on a patch to address this when I saw your unmerged new_pipeline branch which appears to address this same issues, along with some other items.\r\n\r\nI am curious as to the status of this branch and what if anything you see as remaining before it can be called complete.  As we have an immediate need for a modified algorithm, I am happy to help close out this feature branch or work to separate out the specific algorithm changes from the others.  Please let me know what I can do to help drive this to completion.\r\n\r\nI am also wondering about your choice to use a timestamp as part of the key-tuple for connection lookup in ets.  Are you looking for order of creation to be preserved if two connections have the same speculative size?  In our usage, we have a max sessions size of ~512 for traffic between our servers (network load balancer inbetween) and are seeing a significant skew of load (e.g. the first server in a 3 wide cluster gets 10-20% more load than the second, which gets 10-20% more load than the first).  I believe I have traced this back to the repeated offering of work to the first connection before offering to others.  I would like to find a way to more evenly distribute requests across connections with the same pipeline depth in order to ensure the long running connections don't skew overtime.","created_at":"2014-11-17T16:03:45Z","updated_at":"2014-11-17T16:03:45Z","closed_at":null,"merged_at":null,"merge_commit_sha":null,"assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/cmullaparthi/ibrowse/pulls/122/commits","review_comments_url":"https://api.github.com/repos/cmullaparthi/ibrowse/pulls/122/comments","review_comment_url":"https://api.github.com/repos/cmullaparthi/ibrowse/pulls/comments/{number}","comments_url":"https://api.github.com/repos/cmullaparthi/ibrowse/issues/122/comments","statuses_url":"https://api.github.com/repos/cmullaparthi/ibrowse/statuses/d61dd9a074ac7f9d63445ac9a19670a11aaf9640","head":{"label":"cmullaparthi:new_pipeline","ref":"new_pipeline","sha":"d61dd9a074ac7f9d63445ac9a19670a11aaf9640","user":{"login":"cmullaparthi","id":49323,"avatar_url":"https://avatars.githubusercontent.com/u/49323?v=3","gravatar_id":"","url":"https://api.github.com/users/cmullaparthi","html_url":"https://github.com/cmullaparthi","followers_url":"https://api.github.com/users/cmullaparthi/followers","following_url":"https://api.github.com/users/cmullaparthi/following{/other_user}","gists_url":"https://api.github.com/users/cmullaparthi/gists{/gist_id}","starred_url":"https://api.github.com/users/cmullaparthi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmullaparthi/subscriptions","organizations_url":"https://api.github.com/users/cmullaparthi/orgs","repos_url":"https://api.github.com/users/cmullaparthi/repos","events_url":"https://api.github.com/users/cmullaparthi/events{/privacy}","received_events_url":"https://api.github.com/users/cmullaparthi/received_events","type":"User","site_admin":false},"repo":{"id":231268,"name":"ibrowse","full_name":"cmullaparthi/ibrowse","owner":{"login":"cmullaparthi","id":49323,"avatar_url":"https://avatars.githubusercontent.com/u/49323?v=3","gravatar_id":"","url":"https://api.github.com/users/cmullaparthi","html_url":"https://github.com/cmullaparthi","followers_url":"https://api.github.com/users/cmullaparthi/followers","following_url":"https://api.github.com/users/cmullaparthi/following{/other_user}","gists_url":"https://api.github.com/users/cmullaparthi/gists{/gist_id}","starred_url":"https://api.github.com/users/cmullaparthi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmullaparthi/subscriptions","organizations_url":"https://api.github.com/users/cmullaparthi/orgs","repos_url":"https://api.github.com/users/cmullaparthi/repos","events_url":"https://api.github.com/users/cmullaparthi/events{/privacy}","received_events_url":"https://api.github.com/users/cmullaparthi/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/cmullaparthi/ibrowse","description":"Erlang HTTP client","fork":false,"url":"https://api.github.com/repos/cmullaparthi/ibrowse","forks_url":"https://api.github.com/repos/cmullaparthi/ibrowse/forks","keys_url":"https://api.github.com/repos/cmullaparthi/ibrowse/keys{/key_id}","collaborators_url":"https://api.github.com/repos/cmullaparthi/ibrowse/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/cmullaparthi/ibrowse/teams","hooks_url":"https://api.github.com/repos/cmullaparthi/ibrowse/hooks","issue_events_url":"https://api.github.com/repos/cmullaparthi/ibrowse/issues/events{/number}","events_url":"https://api.github.com/repos/cmullaparthi/ibrowse/events","assignees_url":"https://api.github.com/repos/cmullaparthi/ibrowse/assignees{/user}","branches_url":"https://api.github.com/repos/cmullaparthi/ibrowse/branches{/branch}","tags_url":"https://api.github.com/repos/cmullaparthi/ibrowse/tags","blobs_url":"https://api.github.com/repos/cmullaparthi/ibrowse/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/cmullaparthi/ibrowse/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/cmullaparthi/ibrowse/git/refs{/sha}","trees_url":"https://api.github.com/repos/cmullaparthi/ibrowse/git/trees{/sha}","statuses_url":"https://api.github.com/repos/cmullaparthi/ibrowse/statuses/{sha}","languages_url":"https://api.github.com/repos/cmullaparthi/ibrowse/languages","stargazers_url":"https://api.github.com/repos/cmullaparthi/ibrowse/stargazers","contributors_url":"https://api.github.com/repos/cmullaparthi/ibrowse/contributors","subscribers_url":"https://api.github.com/repos/cmullaparthi/ibrowse/subscribers","subscription_url":"https://api.github.com/repos/cmullaparthi/ibrowse/subscription","commits_url":"https://api.github.com/repos/cmullaparthi/ibrowse/commits{/sha}","git_commits_url":"https://api.github.com/repos/cmullaparthi/ibrowse/git/commits{/sha}","comments_url":"https://api.github.com/repos/cmullaparthi/ibrowse/comments{/number}","issue_comment_url":"https://api.github.com/repos/cmullaparthi/ibrowse/issues/comments/{number}","contents_url":"https://api.github.com/repos/cmullaparthi/ibrowse/contents/{+path}","compare_url":"https://api.github.com/repos/cmullaparthi/ibrowse/compare/{base}...{head}","merges_url":"https://api.github.com/repos/cmullaparthi/ibrowse/merges","archive_url":"https://api.github.com/repos/cmullaparthi/ibrowse/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/cmullaparthi/ibrowse/downloads","issues_url":"https://api.github.com/repos/cmullaparthi/ibrowse/issues{/number}","pulls_url":"https://api.github.com/repos/cmullaparthi/ibrowse/pulls{/number}","milestones_url":"https://api.github.com/repos/cmullaparthi/ibrowse/milestones{/number}","notifications_url":"https://api.github.com/repos/cmullaparthi/ibrowse/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/cmullaparthi/ibrowse/labels{/name}","releases_url":"https://api.github.com/repos/cmullaparthi/ibrowse/releases{/id}","created_at":"2009-06-19T09:41:04Z","updated_at":"2014-11-14T20:03:37Z","pushed_at":"2014-08-19T18:05:34Z","git_url":"git://github.com/cmullaparthi/ibrowse.git","ssh_url":"git@github.com:cmullaparthi/ibrowse.git","clone_url":"https://github.com/cmullaparthi/ibrowse.git","svn_url":"https://github.com/cmullaparthi/ibrowse","homepage":"","size":1908,"stargazers_count":319,"watchers_count":319,"language":"Erlang","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":127,"mirror_url":null,"open_issues_count":9,"forks":127,"open_issues":9,"watchers":319,"default_branch":"master"}},"base":{"label":"cmullaparthi:master","ref":"master","sha":"e18ea912b62b0f4a48d0f0edc8bb2f542863a88c","user":{"login":"cmullaparthi","id":49323,"avatar_url":"https://avatars.githubusercontent.com/u/49323?v=3","gravatar_id":"","url":"https://api.github.com/users/cmullaparthi","html_url":"https://github.com/cmullaparthi","followers_url":"https://api.github.com/users/cmullaparthi/followers","following_url":"https://api.github.com/users/cmullaparthi/following{/other_user}","gists_url":"https://api.github.com/users/cmullaparthi/gists{/gist_id}","starred_url":"https://api.github.com/users/cmullaparthi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmullaparthi/subscriptions","organizations_url":"https://api.github.com/users/cmullaparthi/orgs","repos_url":"https://api.github.com/users/cmullaparthi/repos","events_url":"https://api.github.com/users/cmullaparthi/events{/privacy}","received_events_url":"https://api.github.com/users/cmullaparthi/received_events","type":"User","site_admin":false},"repo":{"id":231268,"name":"ibrowse","full_name":"cmullaparthi/ibrowse","owner":{"login":"cmullaparthi","id":49323,"avatar_url":"https://avatars.githubusercontent.com/u/49323?v=3","gravatar_id":"","url":"https://api.github.com/users/cmullaparthi","html_url":"https://github.com/cmullaparthi","followers_url":"https://api.github.com/users/cmullaparthi/followers","following_url":"https://api.github.com/users/cmullaparthi/following{/other_user}","gists_url":"https://api.github.com/users/cmullaparthi/gists{/gist_id}","starred_url":"https://api.github.com/users/cmullaparthi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmullaparthi/subscriptions","organizations_url":"https://api.github.com/users/cmullaparthi/orgs","repos_url":"https://api.github.com/users/cmullaparthi/repos","events_url":"https://api.github.com/users/cmullaparthi/events{/privacy}","received_events_url":"https://api.github.com/users/cmullaparthi/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/cmullaparthi/ibrowse","description":"Erlang HTTP client","fork":false,"url":"https://api.github.com/repos/cmullaparthi/ibrowse","forks_url":"https://api.github.com/repos/cmullaparthi/ibrowse/forks","keys_url":"https://api.github.com/repos/cmullaparthi/ibrowse/keys{/key_id}","collaborators_url":"https://api.github.com/repos/cmullaparthi/ibrowse/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/cmullaparthi/ibrowse/teams","hooks_url":"https://api.github.com/repos/cmullaparthi/ibrowse/hooks","issue_events_url":"https://api.github.com/repos/cmullaparthi/ibrowse/issues/events{/number}","events_url":"https://api.github.com/repos/cmullaparthi/ibrowse/events","assignees_url":"https://api.github.com/repos/cmullaparthi/ibrowse/assignees{/user}","branches_url":"https://api.github.com/repos/cmullaparthi/ibrowse/branches{/branch}","tags_url":"https://api.github.com/repos/cmullaparthi/ibrowse/tags","blobs_url":"https://api.github.com/repos/cmullaparthi/ibrowse/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/cmullaparthi/ibrowse/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/cmullaparthi/ibrowse/git/refs{/sha}","trees_url":"https://api.github.com/repos/cmullaparthi/ibrowse/git/trees{/sha}","statuses_url":"https://api.github.com/repos/cmullaparthi/ibrowse/statuses/{sha}","languages_url":"https://api.github.com/repos/cmullaparthi/ibrowse/languages","stargazers_url":"https://api.github.com/repos/cmullaparthi/ibrowse/stargazers","contributors_url":"https://api.github.com/repos/cmullaparthi/ibrowse/contributors","subscribers_url":"https://api.github.com/repos/cmullaparthi/ibrowse/subscribers","subscription_url":"https://api.github.com/repos/cmullaparthi/ibrowse/subscription","commits_url":"https://api.github.com/repos/cmullaparthi/ibrowse/commits{/sha}","git_commits_url":"https://api.github.com/repos/cmullaparthi/ibrowse/git/commits{/sha}","comments_url":"https://api.github.com/repos/cmullaparthi/ibrowse/comments{/number}","issue_comment_url":"https://api.github.com/repos/cmullaparthi/ibrowse/issues/comments/{number}","contents_url":"https://api.github.com/repos/cmullaparthi/ibrowse/contents/{+path}","compare_url":"https://api.github.com/repos/cmullaparthi/ibrowse/compare/{base}...{head}","merges_url":"https://api.github.com/repos/cmullaparthi/ibrowse/merges","archive_url":"https://api.github.com/repos/cmullaparthi/ibrowse/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/cmullaparthi/ibrowse/downloads","issues_url":"https://api.github.com/repos/cmullaparthi/ibrowse/issues{/number}","pulls_url":"https://api.github.com/repos/cmullaparthi/ibrowse/pulls{/number}","milestones_url":"https://api.github.com/repos/cmullaparthi/ibrowse/milestones{/number}","notifications_url":"https://api.github.com/repos/cmullaparthi/ibrowse/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/cmullaparthi/ibrowse/labels{/name}","releases_url":"https://api.github.com/repos/cmullaparthi/ibrowse/releases{/id}","created_at":"2009-06-19T09:41:04Z","updated_at":"2014-11-14T20:03:37Z","pushed_at":"2014-08-19T18:05:34Z","git_url":"git://github.com/cmullaparthi/ibrowse.git","ssh_url":"git@github.com:cmullaparthi/ibrowse.git","clone_url":"https://github.com/cmullaparthi/ibrowse.git","svn_url":"https://github.com/cmullaparthi/ibrowse","homepage":"","size":1908,"stargazers_count":319,"watchers_count":319,"language":"Erlang","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":127,"mirror_url":null,"open_issues_count":9,"forks":127,"open_issues":9,"watchers":319,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/cmullaparthi/ibrowse/pulls/122"},"html":{"href":"https://github.com/cmullaparthi/ibrowse/pull/122"},"issue":{"href":"https://api.github.com/repos/cmullaparthi/ibrowse/issues/122"},"comments":{"href":"https://api.github.com/repos/cmullaparthi/ibrowse/issues/122/comments"},"review_comments":{"href":"https://api.github.com/repos/cmullaparthi/ibrowse/pulls/122/comments"},"review_comment":{"href":"https://api.github.com/repos/cmullaparthi/ibrowse/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/cmullaparthi/ibrowse/pulls/122/commits"},"statuses":{"href":"https://api.github.com/repos/cmullaparthi/ibrowse/statuses/d61dd9a074ac7f9d63445ac9a19670a11aaf9640"}},"merged":false,"mergeable":null,"mergeable_state":"unknown","merged_by":null,"comments":0,"review_comments":0,"commits":14,"additions":638,"deletions":267,"changed_files":9}}