clojurewerkz/elastisch — pull request #123 by puszczyk (2014-10-21 08:02:25+00:00)
Title: Optional params implementation

This should fix Issue #122

I've added new update-with-script version to maintain backward compatibility. There are three tests two to prove the backward compatibility and one to add the optional params.

The tests are elasticsearch version dependent. I've run them with success on elasticsearch-1.1.0 with `lang-groovy`, but I had to comment first two to run them in elasticsearch-1.3.2. In order not to break CI I've added new profile `:version-dependent` to exclude the tests from CI (we do not control es versions and plugins).

The reason why the default two versions do not work on 1.2+ is that elasticsearch disabled scrip execution by default, except *sandboxed* scripts. Groovy is considered sandboxed, but mvel not. Groovy does not need a plugin on 1.3+ since its included in the es core.

Please let me know if it makes sense.