{"action":"created","issue":{"url":"https://api.github.com/repos/flux-framework/flux-core/issues/298","labels_url":"https://api.github.com/repos/flux-framework/flux-core/issues/298/labels{/name}","comments_url":"https://api.github.com/repos/flux-framework/flux-core/issues/298/comments","events_url":"https://api.github.com/repos/flux-framework/flux-core/issues/298/events","html_url":"https://github.com/flux-framework/flux-core/pull/298","id":97589130,"number":298,"title":"revive the 'mon' module from ngrm repo","user":{"login":"garlick","id":169947,"avatar_url":"https://avatars.githubusercontent.com/u/169947?v=3","gravatar_id":"","url":"https://api.github.com/users/garlick","html_url":"https://github.com/garlick","followers_url":"https://api.github.com/users/garlick/followers","following_url":"https://api.github.com/users/garlick/following{/other_user}","gists_url":"https://api.github.com/users/garlick/gists{/gist_id}","starred_url":"https://api.github.com/users/garlick/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garlick/subscriptions","organizations_url":"https://api.github.com/users/garlick/orgs","repos_url":"https://api.github.com/users/garlick/repos","events_url":"https://api.github.com/users/garlick/events{/privacy}","received_events_url":"https://api.github.com/users/garlick/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/flux-framework/flux-core/labels/review","name":"review","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-07-28T00:53:03Z","updated_at":"2015-07-29T18:52:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/flux-framework/flux-core/pulls/298","html_url":"https://github.com/flux-framework/flux-core/pull/298","diff_url":"https://github.com/flux-framework/flux-core/pull/298.diff","patch_url":"https://github.com/flux-framework/flux-core/pull/298.patch"},"body":"This PR is preliminary, not for merging.  It's posted as a PR mainly to keep @dongahn up to date with what I'm doing to stuff his work depends on.  (He should feel free to redirect me if I am making life harder rather than easier):\r\n* Rename `flux_red_t` to `flux_reduce_t` and make opaque type conformant to flux style\r\n* Rename functions to be prefixed with `flux_reduce_`\r\n* Drop `flux_redstack_t` and add a standalone `flux_list_t` class instead.\r\n* Revive the `flux-mon` command and `mon` module (update to modern interfaces)\r\n* Update the live module to use the new `flux_reduce_t` names.\r\n\r\nSee comment at the top of `src/modules/mon/mon.c` for how to demo the mon module/program.\r\n\r\nComing soon: man page for reduce functions."},"comment":{"url":"https://api.github.com/repos/flux-framework/flux-core/issues/comments/126056631","html_url":"https://github.com/flux-framework/flux-core/pull/298#issuecomment-126056631","issue_url":"https://api.github.com/repos/flux-framework/flux-core/issues/298","id":126056631,"user":{"login":"dongahn","id":862123,"avatar_url":"https://avatars.githubusercontent.com/u/862123?v=3","gravatar_id":"","url":"https://api.github.com/users/dongahn","html_url":"https://github.com/dongahn","followers_url":"https://api.github.com/users/dongahn/followers","following_url":"https://api.github.com/users/dongahn/following{/other_user}","gists_url":"https://api.github.com/users/dongahn/gists{/gist_id}","starred_url":"https://api.github.com/users/dongahn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dongahn/subscriptions","organizations_url":"https://api.github.com/users/dongahn/orgs","repos_url":"https://api.github.com/users/dongahn/repos","events_url":"https://api.github.com/users/dongahn/events{/privacy}","received_events_url":"https://api.github.com/users/dongahn/received_events","type":"User","site_admin":false},"created_at":"2015-07-29T18:52:13Z","updated_at":"2015-07-29T18:52:13Z","body":"The changes look good to me. Adjusting the named reduction to that changes should be straightforward. A question what is the \"register callbacks at creation\" support though? Is this to hide communication needed for reduction from the users?\r\n\r\n>\r\ntransparently installs a request handler so that passing messages upstream can happen behind the scenes\r\n\r\nIn any case, I think hiding communication details for reduction is an excellent direction to pursue. \r\n"}}