{"action":"created","issue":{"url":"https://api.github.com/repos/cockroachdb/loadgen/issues/137","repository_url":"https://api.github.com/repos/cockroachdb/loadgen","labels_url":"https://api.github.com/repos/cockroachdb/loadgen/issues/137/labels{/name}","comments_url":"https://api.github.com/repos/cockroachdb/loadgen/issues/137/comments","events_url":"https://api.github.com/repos/cockroachdb/loadgen/issues/137/events","html_url":"https://github.com/cockroachdb/loadgen/pull/137","id":301554124,"number":137,"title":"tpcc: batch all writes in delivery txn","user":{"login":"nvanbenschoten","id":5438456,"avatar_url":"https://avatars3.githubusercontent.com/u/5438456?v=4","gravatar_id":"","url":"https://api.github.com/users/nvanbenschoten","html_url":"https://github.com/nvanbenschoten","followers_url":"https://api.github.com/users/nvanbenschoten/followers","following_url":"https://api.github.com/users/nvanbenschoten/following{/other_user}","gists_url":"https://api.github.com/users/nvanbenschoten/gists{/gist_id}","starred_url":"https://api.github.com/users/nvanbenschoten/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nvanbenschoten/subscriptions","organizations_url":"https://api.github.com/users/nvanbenschoten/orgs","repos_url":"https://api.github.com/users/nvanbenschoten/repos","events_url":"https://api.github.com/users/nvanbenschoten/events{/privacy}","received_events_url":"https://api.github.com/users/nvanbenschoten/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-03-01T20:48:29Z","updated_at":"2018-03-02T19:32:26Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/cockroachdb/loadgen/pulls/137","html_url":"https://github.com/cockroachdb/loadgen/pull/137","diff_url":"https://github.com/cockroachdb/loadgen/pull/137.diff","patch_url":"https://github.com/cockroachdb/loadgen/pull/137.patch"},"body":"Alternative to #135.\r\n\r\nThis change replaces the for-each-district loop in the delivery\r\ntransaction with a series of batched statements that perform all\r\nwrites together. This reduces the number of round-trips required\r\nby the transaction by an order of magnitude."},"comment":{"url":"https://api.github.com/repos/cockroachdb/loadgen/issues/comments/370028314","html_url":"https://github.com/cockroachdb/loadgen/pull/137#issuecomment-370028314","issue_url":"https://api.github.com/repos/cockroachdb/loadgen/issues/137","id":370028314,"user":{"login":"nvanbenschoten","id":5438456,"avatar_url":"https://avatars3.githubusercontent.com/u/5438456?v=4","gravatar_id":"","url":"https://api.github.com/users/nvanbenschoten","html_url":"https://github.com/nvanbenschoten","followers_url":"https://api.github.com/users/nvanbenschoten/followers","following_url":"https://api.github.com/users/nvanbenschoten/following{/other_user}","gists_url":"https://api.github.com/users/nvanbenschoten/gists{/gist_id}","starred_url":"https://api.github.com/users/nvanbenschoten/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nvanbenschoten/subscriptions","organizations_url":"https://api.github.com/users/nvanbenschoten/orgs","repos_url":"https://api.github.com/users/nvanbenschoten/repos","events_url":"https://api.github.com/users/nvanbenschoten/events{/privacy}","received_events_url":"https://api.github.com/users/nvanbenschoten/received_events","type":"User","site_admin":false},"created_at":"2018-03-02T19:32:26Z","updated_at":"2018-03-02T19:32:26Z","author_association":"OWNER","body":"> The pMax of the other operations are worse on this branch than the other. I think this branch improves the delivery latency at the expense of other transactions.\r\n\r\nDoesn't the improvement in `p90/p95` latencies across all operations indicate the opposite? I don't really think `pMax` is what we should be comparing here."}}
{"actor":{"display_login":"nvanbenschoten"}}