{"action":"created","issue":{"url":"https://api.github.com/repos/jackdoe/bzzz/issues/1","labels_url":"https://api.github.com/repos/jackdoe/bzzz/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/jackdoe/bzzz/issues/1/comments","events_url":"https://api.github.com/repos/jackdoe/bzzz/issues/1/events","html_url":"https://github.com/jackdoe/bzzz/pull/1","id":54621094,"number":1,"title":"RFC: WIP: JSON endpoint for modifying the global read-only state","user":{"login":"caio","id":1898,"avatar_url":"https://avatars.githubusercontent.com/u/1898?v=3","gravatar_id":"","url":"https://api.github.com/users/caio","html_url":"https://github.com/caio","followers_url":"https://api.github.com/users/caio/followers","following_url":"https://api.github.com/users/caio/following{/other_user}","gists_url":"https://api.github.com/users/caio/gists{/gist_id}","starred_url":"https://api.github.com/users/caio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/caio/subscriptions","organizations_url":"https://api.github.com/users/caio/orgs","repos_url":"https://api.github.com/users/caio/repos","events_url":"https://api.github.com/users/caio/events{/privacy}","received_events_url":"https://api.github.com/users/caio/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-01-16T20:27:12Z","updated_at":"2015-01-18T11:20:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jackdoe/bzzz/pulls/1","html_url":"https://github.com/jackdoe/bzzz/pull/1","diff_url":"https://github.com/jackdoe/bzzz/pull/1.diff","patch_url":"https://github.com/jackdoe/bzzz/pull/1.patch"},"body":"Oy,\r\n\r\nThis branch allows us to play with `EXPR_GLOBAL_STATE_RO` outside of the `search-many` context. There is plenty I'm not really happy with yet (expressed as `FIXME` comments in the files):\r\n\r\n 1. We replace the global state every single time. If we are pumping too much data, this is less than ideal. Still need to check if it's worth to use the read-write state as a temporary store\r\n\r\n 2. Clojure's hash-map to java.util.HashMap everywhere. It's rather silly. We should either completely remove it or do a proper conversion (including values).\r\n\r\n 3. Using `GET` because lazy :lollipop: \r\n\r\nAnyway, just getting the conversation started, I don't expect this to be merged as-is. Bash away! Also please bash my clojure-fu, I'm learning it as I write this :-)\r\n\r\nCheers"},"comment":{"url":"https://api.github.com/repos/jackdoe/bzzz/issues/comments/70404329","html_url":"https://github.com/jackdoe/bzzz/pull/1#issuecomment-70404329","issue_url":"https://api.github.com/repos/jackdoe/bzzz/issues/1","id":70404329,"user":{"login":"caio","id":1898,"avatar_url":"https://avatars.githubusercontent.com/u/1898?v=3","gravatar_id":"","url":"https://api.github.com/users/caio","html_url":"https://github.com/caio","followers_url":"https://api.github.com/users/caio/followers","following_url":"https://api.github.com/users/caio/following{/other_user}","gists_url":"https://api.github.com/users/caio/gists{/gist_id}","starred_url":"https://api.github.com/users/caio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/caio/subscriptions","organizations_url":"https://api.github.com/users/caio/orgs","repos_url":"https://api.github.com/users/caio/repos","events_url":"https://api.github.com/users/caio/events{/privacy}","received_events_url":"https://api.github.com/users/caio/received_events","type":"User","site_admin":false},"created_at":"2015-01-18T11:20:10Z","updated_at":"2015-01-18T11:20:10Z","body":"Cool, I'm merging then! I've made it not copy into an empty map on 16b79262 (it was to workaround an issue, I \"fixed\" it differently now). Without this patch a `rename-keys` in the original RO state throws an exception.\r\n\r\nTo-do:\r\n\r\n * Endpoint to transform the data via `clj-eval`\r\n * Put a giant lock for `temporary-state`"}}