{"action":"created","issue":{"url":"https://api.github.com/repos/elastic/elasticsearch/issues/34270","repository_url":"https://api.github.com/repos/elastic/elasticsearch","labels_url":"https://api.github.com/repos/elastic/elasticsearch/issues/34270/labels{/name}","comments_url":"https://api.github.com/repos/elastic/elasticsearch/issues/34270/comments","events_url":"https://api.github.com/repos/elastic/elasticsearch/issues/34270/events","html_url":"https://github.com/elastic/elasticsearch/pull/34270","id":366341750,"node_id":"MDExOlB1bGxSZXF1ZXN0MjIwMDM5MTkw","number":34270,"title":"Tasks: Document that status is not semvered","user":{"login":"nik9000","id":215970,"node_id":"MDQ6VXNlcjIxNTk3MA==","avatar_url":"https://avatars2.githubusercontent.com/u/215970?v=4","gravatar_id":"","url":"https://api.github.com/users/nik9000","html_url":"https://github.com/nik9000","followers_url":"https://api.github.com/users/nik9000/followers","following_url":"https://api.github.com/users/nik9000/following{/other_user}","gists_url":"https://api.github.com/users/nik9000/gists{/gist_id}","starred_url":"https://api.github.com/users/nik9000/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nik9000/subscriptions","organizations_url":"https://api.github.com/users/nik9000/orgs","repos_url":"https://api.github.com/users/nik9000/repos","events_url":"https://api.github.com/users/nik9000/events{/privacy}","received_events_url":"https://api.github.com/users/nik9000/received_events","type":"User","site_admin":false},"labels":[{"id":289830280,"node_id":"MDU6TGFiZWwyODk4MzAyODA=","url":"https://api.github.com/repos/elastic/elasticsearch/labels/:Distributed/Task%20Management","name":":Distributed/Task Management","color":"0e8a16","default":false},{"id":94283388,"node_id":"MDU6TGFiZWw5NDI4MzM4OA==","url":"https://api.github.com/repos/elastic/elasticsearch/labels/review","name":"review","color":"0052cc","default":false},{"id":962378555,"node_id":"MDU6TGFiZWw5NjIzNzg1NTU=","url":"https://api.github.com/repos/elastic/elasticsearch/labels/v6.5.0","name":"v6.5.0","color":"Dddddd","default":false},{"id":484855464,"node_id":"MDU6TGFiZWw0ODQ4NTU0NjQ=","url":"https://api.github.com/repos/elastic/elasticsearch/labels/v7.0.0","name":"v7.0.0","color":"dddddd","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2018-10-03T13:34:49Z","updated_at":"2018-10-03T17:38:29Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/elastic/elasticsearch/pulls/34270","html_url":"https://github.com/elastic/elasticsearch/pull/34270","diff_url":"https://github.com/elastic/elasticsearch/pull/34270.diff","patch_url":"https://github.com/elastic/elasticsearch/pull/34270.patch"},"body":"The `status` part of the tasks API reflects the internal status of a\r\nrunning task. In general, we do not make backwards breaking changes to\r\nthe `status` but because it is internal we reserve the right to do so. I\r\nsuspect we will very rarely excercise that right but it is important\r\nthat we have it so we're not boxed into any particular implementation\r\nfor a request.\r\n\r\nIn some sense this is policy making by documentation change. In another\r\nit is clarification of the way we've always thought of this field.\r\n\r\nI also reflect the documentation change into the Javadoc in a few\r\nplaces. There I acknowledge Kibana's \"special relationship\" with\r\nElasticsearch. Kibana parses `_reindex`'s `status` field and, because\r\nwe're friends with those folks, we should talk to them before we make\r\nbackwards breaking changes to it. We *want* to be friends with everyone\r\nbut there is only so much time in the day and we don't *want* to make\r\nbackwards breaking fields to `status` at all anyway. So we hope that\r\nbreaking changes documentation should be enough for other folks.\r\n\r\nRelates to #34245.\r\n"},"comment":{"url":"https://api.github.com/repos/elastic/elasticsearch/issues/comments/426728647","html_url":"https://github.com/elastic/elasticsearch/pull/34270#issuecomment-426728647","issue_url":"https://api.github.com/repos/elastic/elasticsearch/issues/34270","id":426728647,"node_id":"MDEyOklzc3VlQ29tbWVudDQyNjcyODY0Nw==","user":{"login":"nik9000","id":215970,"node_id":"MDQ6VXNlcjIxNTk3MA==","avatar_url":"https://avatars2.githubusercontent.com/u/215970?v=4","gravatar_id":"","url":"https://api.github.com/users/nik9000","html_url":"https://github.com/nik9000","followers_url":"https://api.github.com/users/nik9000/followers","following_url":"https://api.github.com/users/nik9000/following{/other_user}","gists_url":"https://api.github.com/users/nik9000/gists{/gist_id}","starred_url":"https://api.github.com/users/nik9000/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nik9000/subscriptions","organizations_url":"https://api.github.com/users/nik9000/orgs","repos_url":"https://api.github.com/users/nik9000/repos","events_url":"https://api.github.com/users/nik9000/events{/privacy}","received_events_url":"https://api.github.com/users/nik9000/received_events","type":"User","site_admin":false},"created_at":"2018-10-03T17:38:29Z","updated_at":"2018-10-03T17:38:29Z","author_association":"CONTRIBUTOR","body":"> LGTM, I wonder if we can somehow add a test for that:\r\n\r\nReindex has a test for it.\r\n\r\n@jasontedor, are you ok with making this the official policy?"}}
{"actor":{"display_login":"nik9000"}}