{"action":"opened","issue":{"url":"https://api.github.com/repos/opendatasoft/elasticsearch-aggregation-pathhierarchy/issues/1","labels_url":"https://api.github.com/repos/opendatasoft/elasticsearch-aggregation-pathhierarchy/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/opendatasoft/elasticsearch-aggregation-pathhierarchy/issues/1/comments","events_url":"https://api.github.com/repos/opendatasoft/elasticsearch-aggregation-pathhierarchy/issues/1/events","html_url":"https://github.com/opendatasoft/elasticsearch-aggregation-pathhierarchy/issues/1","id":95961168,"number":1,"title":"Separators that conflict with regex special characters do not work","user":{"login":"byates-va","id":2278954,"avatar_url":"https://avatars.githubusercontent.com/u/2278954?v=3","gravatar_id":"","url":"https://api.github.com/users/byates-va","html_url":"https://github.com/byates-va","followers_url":"https://api.github.com/users/byates-va/followers","following_url":"https://api.github.com/users/byates-va/following{/other_user}","gists_url":"https://api.github.com/users/byates-va/gists{/gist_id}","starred_url":"https://api.github.com/users/byates-va/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/byates-va/subscriptions","organizations_url":"https://api.github.com/users/byates-va/orgs","repos_url":"https://api.github.com/users/byates-va/repos","events_url":"https://api.github.com/users/byates-va/events{/privacy}","received_events_url":"https://api.github.com/users/byates-va/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-07-20T00:14:36Z","updated_at":"2015-07-20T00:14:36Z","closed_at":null,"body":"Hi, first off, I love the idea of this plugin. Until one day hopefully elasticsearch incorporates it as a core aggregation, it is exactly what I need.\r\n\r\nHowever, I have found that our separator, the \"|\" symbol, does not work properly because the separator string is passed straight to the java string split function, which accepts a regex string (the \"|\" would normally need to be escaped).\r\n\r\nPassing the separator already-escaped via the API does not work either; there are other sections of the code that interpret the separator as a regular string, therefore the escape characters become part of the separator in those scenarios.\r\n\r\nMy proposed solution would be to not use the default java string.split, but rather some method that splits on the raw string, so that the input is always simply interpreted as is. \r\n\r\nI can code the solution, but is this something you would be interested in accepting a pull request for so that everybody gets the fix? Or should I simply fork the plugin.\r\n\r\nThanks!"}}