Commits on Aug 11, 2016

1. Make reindex and lang-javascript compatible …

   Fixes two issues:
   1. lang-javascript doesn't support `executable` with a `null` `vars`
   parameters. The parameter is quite nullable.
   2. reindex didn't support script engines who's `unwrap` method wasn't
   a noop. This didn't come up for lang-groovy or lang-painless because
   both of those `unwrap`s were noops. lang-javascript copys all maps that
   it `unwrap`s.
   
   This adds fairly low level unit tests for these fixes but dosen't add
   an integration test that makes sure that reindex and lang-javascript
   play well together. That'd make backporting this difficult and would
   add a fairly significant amount of time to the build for a fairly rare
   interaction. Hopefully the unit tests will be enough.

   

   nik9000 committed Aug 11, 2016
   Configuration menu

   • View commit details
   • Copy full SHA for e07e5d6
   • Browse repository at this point

   Copy the full SHA

   e07e5d6 View commit details

   Browse the repository at this point in the history