{"push_id":1275155993,"size":10,"distinct_size":8,"ref":"refs/heads/kvs_issue_776","head":"e1a5cbffe338039d42b9c844529d35943a355ed7","before":"048ef65167700b64281a4eef836b172281707eba","commits":[{"sha":"7a7f8422dab907f139df78c88bac42c9a5f40369","author":{"name":"Mark A. Grondona","email":"68944fe2049fb4fbc8be4ca877a495307613d028@llnl.gov"},"message":"broker/hello: fix leak/error detection in flux_rpc\n\nThe flux_rpc() use in broker/hello.c seemed to assume integer return\nand tested for error code < 0. Fix the call so that the flux_rpc_t\nreturned from the call is captured, and test for NULL on error.\n\nThis also allows the rpc object to be freed, fixing a small leak.","distinct":false,"url":"https://api.github.com/repos/garlick/flux-core/commits/7a7f8422dab907f139df78c88bac42c9a5f40369"},{"sha":"1e403d93f72b23bec1871c93e12f650c3e07b02f","author":{"name":"Jim Garlick","email":"68f7fa2610b0dc79dc156867f188832fba6e3cfc@gmail.com"},"message":"Merge pull request #786 from grondo/hello-leak\n\nbroker/hello: fix leak/error detection in flux_rpc","distinct":false,"url":"https://api.github.com/repos/garlick/flux-core/commits/1e403d93f72b23bec1871c93e12f650c3e07b02f"},{"sha":"e82e4416b979e887103d925a28814f941de400ef","author":{"name":"Jim Garlick","email":"68f7fa2610b0dc79dc156867f188832fba6e3cfc@gmail.com"},"message":"modules/kvs: restore commit consistency\n\nProblem: KVS \"causal consistency\" and potentially \"read your writes\"\nconsistency are no longer guaranteed since kvs_commit() can return\nbefore committed data has reached the content store.\n\nMake all content store operations from the KVS rank 0 module\nsynchronous.\n\nThis is going to increase commit latency and render the KVS\nunresponsive to other operations while a commit is in progress.\nAddress performance in a later commit.\n\nFixes #776","distinct":true,"url":"https://api.github.com/repos/garlick/flux-core/commits/e82e4416b979e887103d925a28814f941de400ef"},{"sha":"2114c803b3dfe1b40ce150751dc191d93dff5c17","author":{"name":"Jim Garlick","email":"68f7fa2610b0dc79dc156867f188832fba6e3cfc@gmail.com"},"message":"modules/kvs: drop extra sender decode in put handler","distinct":true,"url":"https://api.github.com/repos/garlick/flux-core/commits/2114c803b3dfe1b40ce150751dc191d93dff5c17"},{"sha":"7c1d2ae973905fa31d6f00a6cfad1b18e06862c0","author":{"name":"Jim Garlick","email":"68f7fa2610b0dc79dc156867f188832fba6e3cfc@gmail.com"},"message":"modules/libkvs: use fence to implement commit\n\nImplement kvs_commit() as kvs_fence (name, 1), where name\nis a uuid generated for the single commit.\n\nThis allows commits to complete in the same efficient way\nas fences;  that is, piggybacked on the setroot event\nresulting from the commit.\n\nAdd async interfaces: kvs_commit_begin() and kvs_commit_finish()","distinct":true,"url":"https://api.github.com/repos/garlick/flux-core/commits/7c1d2ae973905fa31d6f00a6cfad1b18e06862c0"},{"sha":"154a034a8fedd4206f3df77484521052d1f7f48d","author":{"name":"Jim Garlick","email":"68f7fa2610b0dc79dc156867f188832fba6e3cfc@gmail.com"},"message":"modules/kvs: drop unnecessary commit handling code\n\nDrop separate handling for commit requests now that commit\nis implemented in terms of fence.","distinct":true,"url":"https://api.github.com/repos/garlick/flux-core/commits/154a034a8fedd4206f3df77484521052d1f7f48d"},{"sha":"5ee9a72b8acc4e352bae350e6ae2d85ae11355fc","author":{"name":"Jim Garlick","email":"68f7fa2610b0dc79dc156867f188832fba6e3cfc@gmail.com"},"message":"modules/kvs: drop excessive debug logging","distinct":true,"url":"https://api.github.com/repos/garlick/flux-core/commits/5ee9a72b8acc4e352bae350e6ae2d85ae11355fc"},{"sha":"1d80c9f7c572c7198536ae6d7131a0958b0ea8e7","author":{"name":"Jim Garlick","email":"68f7fa2610b0dc79dc156867f188832fba6e3cfc@gmail.com"},"message":"modules/kvs: drop commit codec and its tests","distinct":true,"url":"https://api.github.com/repos/garlick/flux-core/commits/1d80c9f7c572c7198536ae6d7131a0958b0ea8e7"},{"sha":"0aca63cd14aaf6548551cddaea467d64de1c3923","author":{"name":"Jim Garlick","email":"68f7fa2610b0dc79dc156867f188832fba6e3cfc@gmail.com"},"message":"modules/kvs: special case \"handler restart\" wait_t\n\nImplement a new, generic wait_create() that takes a callback\nand arg.  When the wait_t usecount reaches zero, the callback\nis called.\n\nThe \"message handler restart\" use case, formerly the only\nsupported use case, is now a special case.  To underscore this,\nthe interfaces were renamed:\n\n  wait_compare_f       => wait_test_msg_f\n  wait_create()        => wait_create_msg_handler ()\n  wait_destroy_match() => wait_destroy_msg ()\n\nAlso, drop the time argument from wait_destroy().  This was\nonly used to time kvs_get(), which is not all that useful.\nDrop \"get\" and other unused counters from KVS statistics.","distinct":true,"url":"https://api.github.com/repos/garlick/flux-core/commits/0aca63cd14aaf6548551cddaea467d64de1c3923"},{"sha":"e1a5cbffe338039d42b9c844529d35943a355ed7","author":{"name":"Jim Garlick","email":"68f7fa2610b0dc79dc156867f188832fba6e3cfc@gmail.com"},"message":"modules/kvs: stall commit completion on store\n\nMake the commit process \"stallable\" so that the KVS module\ncan remain responsive when content store requests are in progress.\nTry to get all content store requests for a commit in flight\nat once, asynchronously, to avoid boxcar effect on RPCs.\n\nWhen a fence/commit becomes ready, it is added to the ctx->ready queue.\nOnly one commit can be active at a time.  When one finishes, or when\none is enqueued and afterwards, the queue contains exactly one entry,\nbegin processing the commit at the head of the queue.\n\nMake commit_apply_fence() idempotent, and then arrange for it\nto be stalled/restarted when\n * the rootdir or any directories required to walk the (changing)\n   name space are not in local cache\n * any new objects are still dirty with respect to the content cache\n\nCommit coalescing is still not (re-)implemented.","distinct":true,"url":"https://api.github.com/repos/garlick/flux-core/commits/e1a5cbffe338039d42b9c844529d35943a355ed7"}]}
{"actor":{"display_login":"garlick"}}