OpenSearch/elasticsearch
Chris Earle 73ba90b863 Disallow {index} being specified at the top level of _xpack/monitoring/_bulk
The general use case is to provide only the {type} to the bulk and even that may not be specified. However,
by setting it up as

/_x-pack/monitoring/{type}/_bulk
/_x-pack/monitoring/{index}/{type}/_bulk

it fails to properly recognize the {type} parameter because the PathTrie that gets generated sees two wildcards
at the same location and the last one specified wins -- {index}. As a result, it's impossible to only set the
{type} without making the PathTrie logic convoluted for a niche use case (a list to try instead of a single path).

his fixes the issue by removing the completely unused option: you can no longer specify {index} outside of
individual bulk indexing operations. If we see a need to bring it back, then we can add it as an API param, but
that is an unusual place for the index field and I do not expect it to be needed that frequently.

Original commit: elastic/x-pack-elasticsearch@40d0d05404
2016-05-13 18:12:04 -04:00
..
license Switch scripts to use bash 2016-05-10 15:07:04 -04:00
qa Fix scripting engines for singular type 2016-05-13 09:29:37 -06:00
x-dev-tools Smoke Testing: Add smoke tester for licensing 2016-04-28 08:47:31 +02:00
x-pack Disallow {index} being specified at the top level of _xpack/monitoring/_bulk 2016-05-13 18:12:04 -04:00
build.gradle Change path separator for Checkstyle suppressions to be Windows compatible 2016-02-05 12:15:47 +01:00
checkstyle_suppressions.xml Remove suppression and implement hashCode 2016-02-09 21:49:13 -05:00