{"action":"created","issue":{"url":"https://api.github.com/repos/elastic/elasticsearch/issues/20623","repository_url":"https://api.github.com/repos/elastic/elasticsearch","labels_url":"https://api.github.com/repos/elastic/elasticsearch/issues/20623/labels{/name}","comments_url":"https://api.github.com/repos/elastic/elasticsearch/issues/20623/comments","events_url":"https://api.github.com/repos/elastic/elasticsearch/issues/20623/events","html_url":"https://github.com/elastic/elasticsearch/pull/20623","id":178481881,"number":20623,"title":"Fix String Concatenation Bug In Painless","user":{"login":"jdconrad","id":2126764,"avatar_url":"https://avatars.githubusercontent.com/u/2126764?v=3","gravatar_id":"","url":"https://api.github.com/users/jdconrad","html_url":"https://github.com/jdconrad","followers_url":"https://api.github.com/users/jdconrad/followers","following_url":"https://api.github.com/users/jdconrad/following{/other_user}","gists_url":"https://api.github.com/users/jdconrad/gists{/gist_id}","starred_url":"https://api.github.com/users/jdconrad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdconrad/subscriptions","organizations_url":"https://api.github.com/users/jdconrad/orgs","repos_url":"https://api.github.com/users/jdconrad/repos","events_url":"https://api.github.com/users/jdconrad/events{/privacy}","received_events_url":"https://api.github.com/users/jdconrad/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/:Plugin%20Lang%20Painless","name":":Plugin Lang Painless","color":"009800"},{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/bug","name":"bug","color":"9e2c2c"},{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/v5.0.0","name":"v5.0.0","color":"dddddd"},{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/v5.1.0","name":"v5.1.0","color":"dddddd"},{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/v6.0.0","name":"v6.0.0","color":"dddddd"}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-09-21T22:52:34Z","updated_at":"2016-09-23T16:35:28Z","closed_at":"2016-09-21T23:10:48Z","pull_request":{"url":"https://api.github.com/repos/elastic/elasticsearch/pulls/20623","html_url":"https://github.com/elastic/elasticsearch/pull/20623","diff_url":"https://github.com/elastic/elasticsearch/pull/20623.diff","patch_url":"https://github.com/elastic/elasticsearch/pull/20623.patch"},"body":"@nik9000 found a bug related to String concatenation with the following query:\r\n\r\n```\r\ncurl -XPOST localhost:9200/_reindex?pretty&workers=5 -d{\r\n    \"source\": {\r\n      \"index\": \"source\"\r\n    },\r\n    \"dest\": {\r\n      \"index\": \"source2\"\r\n    },\r\n    \"script\": {\r\n      \"inline\": \"ctx._id += params.i + '.' + ctx._id\",\r\n      \"params\": {\r\n        \"i\": \"i\"\r\n      }\r\n    }\r\n  }\r\n```\r\n\r\nThis PR fixes the bug which was related to an extra append being called since the assignment node was doing an erroneous check to see if the append was necessary."},"comment":{"url":"https://api.github.com/repos/elastic/elasticsearch/issues/comments/249240634","html_url":"https://github.com/elastic/elasticsearch/pull/20623#issuecomment-249240634","issue_url":"https://api.github.com/repos/elastic/elasticsearch/issues/20623","id":249240634,"user":{"login":"jdconrad","id":2126764,"avatar_url":"https://avatars.githubusercontent.com/u/2126764?v=3","gravatar_id":"","url":"https://api.github.com/users/jdconrad","html_url":"https://github.com/jdconrad","followers_url":"https://api.github.com/users/jdconrad/followers","following_url":"https://api.github.com/users/jdconrad/following{/other_user}","gists_url":"https://api.github.com/users/jdconrad/gists{/gist_id}","starred_url":"https://api.github.com/users/jdconrad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdconrad/subscriptions","organizations_url":"https://api.github.com/users/jdconrad/orgs","repos_url":"https://api.github.com/users/jdconrad/repos","events_url":"https://api.github.com/users/jdconrad/events{/privacy}","received_events_url":"https://api.github.com/users/jdconrad/received_events","type":"User","site_admin":false},"created_at":"2016-09-23T16:35:28Z","updated_at":"2016-09-23T16:35:28Z","body":"@clintongormley Sorry, I always mess up the labels... I was looking for 5.x, but 5.1 makes sense.  It got back ported to both 5.0 and 5.x"}}
{"actor":{"display_login":"jdconrad"}}