search

aparo / pyes

Python connector for ElasticSearch - the pythonic way to use ElasticSearch
BSD 3-Clause "New" or "Revised" License
607 stars 270 forks source link

RegexTermFilter doesn't work, but you've already got the fix in master #506

Open kurtwheeler opened 8 years ago

kurtwheeler commented 8 years ago

Hey so I'm trying to use the RegexTermFilter and I'm getting an error:

2015-12-10 22:41:06,171 Error on: ElasticSearchException(u'QueryParsingException[[logstash-2015.12.10] No filter registered for [regex_term]]; }]',)

According to https://www.bountysource.com/issues/1521992-regextermfilter-error the filter name should be regexp not regex_term.

It looks like @toconnor already fixed it in master here: https://github.com/aparo/pyes/blob/master/pyes/filters.py#L308 with this commit: https://github.com/aparo/pyes/commit/e40ae66e33103e31a007bd2cfbfb98d78fbd246b however the change never made it into the latest release as you can see here: https://github.com/aparo/pyes/blob/f280758b9aee06fb5509a0b29662df883f623fbf/pyes/filters.py#L308

Could you please just release that code to get this working?

Thanks!

underyx commented 8 years ago

Since this release does not seem to be coming anytime soon, you could just install the package from GitHub and use the master branch version: https://underyx.me/2015/11/23/adding-an-unreleased-commit-as-a-dependency

kurtwheeler commented 8 years ago

Thanks @underyx I did so when I opened the issue cause I didn't want to wait :)