{"action":"created","issue":{"url":"https://api.github.com/repos/cargomedia/CM/issues/1699","labels_url":"https://api.github.com/repos/cargomedia/CM/issues/1699/labels{/name}","comments_url":"https://api.github.com/repos/cargomedia/CM/issues/1699/comments","events_url":"https://api.github.com/repos/cargomedia/CM/issues/1699/events","html_url":"https://github.com/cargomedia/CM/pull/1699","id":62712182,"number":1699,"title":"Rework create, update, delete indexes for Elasticsearch","user":{"login":"tomaszdurka","id":113303,"avatar_url":"https://avatars.githubusercontent.com/u/113303?v=3","gravatar_id":"","url":"https://api.github.com/users/tomaszdurka","html_url":"https://github.com/tomaszdurka","followers_url":"https://api.github.com/users/tomaszdurka/followers","following_url":"https://api.github.com/users/tomaszdurka/following{/other_user}","gists_url":"https://api.github.com/users/tomaszdurka/gists{/gist_id}","starred_url":"https://api.github.com/users/tomaszdurka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomaszdurka/subscriptions","organizations_url":"https://api.github.com/users/tomaszdurka/orgs","repos_url":"https://api.github.com/users/tomaszdurka/repos","events_url":"https://api.github.com/users/tomaszdurka/events{/privacy}","received_events_url":"https://api.github.com/users/tomaszdurka/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/cargomedia/CM/labels/selfmade","name":"selfmade","color":"ededed"}],"state":"open","locked":false,"assignee":{"login":"tomaszdurka","id":113303,"avatar_url":"https://avatars.githubusercontent.com/u/113303?v=3","gravatar_id":"","url":"https://api.github.com/users/tomaszdurka","html_url":"https://github.com/tomaszdurka","followers_url":"https://api.github.com/users/tomaszdurka/followers","following_url":"https://api.github.com/users/tomaszdurka/following{/other_user}","gists_url":"https://api.github.com/users/tomaszdurka/gists{/gist_id}","starred_url":"https://api.github.com/users/tomaszdurka/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomaszdurka/subscriptions","organizations_url":"https://api.github.com/users/tomaszdurka/orgs","repos_url":"https://api.github.com/users/tomaszdurka/repos","events_url":"https://api.github.com/users/tomaszdurka/events{/privacy}","received_events_url":"https://api.github.com/users/tomaszdurka/received_events","type":"User","site_admin":false},"milestone":null,"comments":4,"created_at":"2015-03-18T14:48:32Z","updated_at":"2015-03-18T16:40:14Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cargomedia/CM/pulls/1699","html_url":"https://github.com/cargomedia/CM/pull/1699","diff_url":"https://github.com/cargomedia/CM/pull/1699.diff","patch_url":"https://github.com/cargomedia/CM/pull/1699.patch"},"body":null},"comment":{"url":"https://api.github.com/repos/cargomedia/CM/issues/comments/83051124","html_url":"https://github.com/cargomedia/CM/pull/1699#issuecomment-83051124","issue_url":"https://api.github.com/repos/cargomedia/CM/issues/1699","id":83051124,"user":{"login":"njam","id":360800,"avatar_url":"https://avatars.githubusercontent.com/u/360800?v=3","gravatar_id":"","url":"https://api.github.com/users/njam","html_url":"https://github.com/njam","followers_url":"https://api.github.com/users/njam/followers","following_url":"https://api.github.com/users/njam/following{/other_user}","gists_url":"https://api.github.com/users/njam/gists{/gist_id}","starred_url":"https://api.github.com/users/njam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/njam/subscriptions","organizations_url":"https://api.github.com/users/njam/orgs","repos_url":"https://api.github.com/users/njam/repos","events_url":"https://api.github.com/users/njam/events{/privacy}","received_events_url":"https://api.github.com/users/njam/received_events","type":"User","site_admin":false},"created_at":"2015-03-18T16:40:14Z","updated_at":"2015-03-18T16:40:14Z","body":"- It might be good to have `CM_Elasticsearch_Cluster::updateIndex()` and `CM_Elasticsearch_Type_Abstract::updateItem()` in the same file (as they are two sides of the same queue). How about moving both to the type?\r\n- CM_Elasticsearch_Cluster::createIndex: I don't think this should refresh the index. This might be a heavy operation and I wouldn't expect it to happen when calling this API? Can we move it to the usages?\r\n - CM_Elasticsearch_Cluster::createIndex,deleteIndex: Remove both of there?\r\n- CM_Elasticsearch_Index_Cli: Very good to get rid of all this logic from here!\r\n- CM_Elasticsearch_Index_Cli: Introduce a method to return either all types or one based on name?"}}