sherlock
sherlock copied to clipboard
Refactor Filters with Null default parameters
Defaults are hardcoded into filters, basically as best guesses taken from the Perl client. Would be much better to just pass Null and let ES use the internal defaults.
This also reduces maintenance, since ES defaults can change and they will automatically propagate to Sherlock without any changes on our end.
Related to #27
Quick update on this: ES devs say that while Null may work, it is probably not the best solution. There are a few locations (notably "upsert" in the Update operations) where a Null is actually parsed and throws crazy errors.
They advised that parameters only be sent to ES when they are actually set. Which means considerably more work for me :(