{"action":"created","issue":{"url":"https://api.github.com/repos/elastic/elasticsearch/issues/6198","repository_url":"https://api.github.com/repos/elastic/elasticsearch","labels_url":"https://api.github.com/repos/elastic/elasticsearch/issues/6198/labels{/name}","comments_url":"https://api.github.com/repos/elastic/elasticsearch/issues/6198/comments","events_url":"https://api.github.com/repos/elastic/elasticsearch/issues/6198/events","html_url":"https://github.com/elastic/elasticsearch/pull/6198","id":33632261,"number":6198,"title":"Adds toString() functionality to SuggestRequestBuilder + SuggestBuilder","user":{"login":"bly2k","id":3187614,"avatar_url":"https://avatars.githubusercontent.com/u/3187614?v=3","gravatar_id":"","url":"https://api.github.com/users/bly2k","html_url":"https://github.com/bly2k","followers_url":"https://api.github.com/users/bly2k/followers","following_url":"https://api.github.com/users/bly2k/following{/other_user}","gists_url":"https://api.github.com/users/bly2k/gists{/gist_id}","starred_url":"https://api.github.com/users/bly2k/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bly2k/subscriptions","organizations_url":"https://api.github.com/users/bly2k/orgs","repos_url":"https://api.github.com/users/bly2k/repos","events_url":"https://api.github.com/users/bly2k/events{/privacy}","received_events_url":"https://api.github.com/users/bly2k/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/:Suggesters","name":":Suggesters","color":"009800"},{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/enhancement","name":"enhancement","color":"4a4ea8"},{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/feedback_needed","name":"feedback_needed","color":"d4c5f9"},{"url":"https://api.github.com/repos/elastic/elasticsearch/labels/review","name":"review","color":"0052cc"}],"state":"open","locked":false,"assignee":{"login":"areek","id":753679,"avatar_url":"https://avatars.githubusercontent.com/u/753679?v=3","gravatar_id":"","url":"https://api.github.com/users/areek","html_url":"https://github.com/areek","followers_url":"https://api.github.com/users/areek/followers","following_url":"https://api.github.com/users/areek/following{/other_user}","gists_url":"https://api.github.com/users/areek/gists{/gist_id}","starred_url":"https://api.github.com/users/areek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/areek/subscriptions","organizations_url":"https://api.github.com/users/areek/orgs","repos_url":"https://api.github.com/users/areek/repos","events_url":"https://api.github.com/users/areek/events{/privacy}","received_events_url":"https://api.github.com/users/areek/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2014-05-15T22:48:38Z","updated_at":"2016-02-12T15:55:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/elastic/elasticsearch/pulls/6198","html_url":"https://github.com/elastic/elasticsearch/pull/6198","diff_url":"https://github.com/elastic/elasticsearch/pull/6198.diff","patch_url":"https://github.com/elastic/elasticsearch/pull/6198.patch"},"body":"Just like SearchRequestBuilder, this adds toString() functionality to SuggestRequestBuilder + SuggestBuilder"},"comment":{"url":"https://api.github.com/repos/elastic/elasticsearch/issues/comments/183384451","html_url":"https://github.com/elastic/elasticsearch/pull/6198#issuecomment-183384451","issue_url":"https://api.github.com/repos/elastic/elasticsearch/issues/6198","id":183384451,"user":{"login":"nik9000","id":215970,"avatar_url":"https://avatars.githubusercontent.com/u/215970?v=3","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":"2016-02-12T15:55:25Z","updated_at":"2016-02-12T15:55:25Z","body":"I think the whole \"helper method for an xcontent toString\" is a good idea. I see us repeat it quite a bit. Anyone involved in the query refactoring like the idea? @abeyad or @colings86?\r\n\r\nPersonally I'd just do the wrapInObject=true case.\r\n\r\nIs it too crazy to implement it with a default method on an interface that extends ToXContent? Heck, it could even be ***on*** ToXContent if we limit the length of the result.\r\n"}}