Added stopwords: _none_ to the docs #329

[mrkamel]

I'm missing stopwords: _none_ in the docs, as well as a stopwords: ... section within the pattern analyzer docs.

[clintongormley]

Hi @mrkamel

Many thanks for this PR. Reading through it made me realise that we have some inconsistent changes coming up. In 1.0 we have set the default stopwords list on new indices for the standard analyzer to _none_ (which apparently isn't documented yet).

I've just opened #4699 to get the pattern and standard_html analyzers changed to default to _none_. We should probably wait for that before moving forward with this PR.

Also, please could you sign the CLA http://www.elasticsearch.org/contributor-agreement so that we can accept these changes.

many thanks

[mrkamel]

a) The docs of master state

A list of stopwords to initialize the stop filter with.
Defaults to an 'empty' stopword list added[1.0.0.Beta1, Previously 
defaulted to the English stopwords list]

b) I agree, we should wait for the removal of inconsistencies
c) Signed
d) You're right, it's probably not the best page to add this section. However, i used it since

        ...
        stopwords : [test1, test2, test3]

was already listed in the page's sample config. Thus, i assume stopwords should not be listed on this page at all and we should provide an extra page for them to have less duplication in the docs and then reference it where neccessary. Where do you want it to live? analysis/analyzers/stopwords.asciidoc?

[mrkamel]

Great, #4699 seems to be fixed. Do you agree to my proposal in d) ?
I'd create another pull request then ... and close this one

[clintongormley]

Heya

Re:
(a) - I missed that :)
(b) done, we won't document StandardHtml
(c) Great, thanks!
(d) I would:

• leave the stopwords example as it was originally on the analyzers page - it's just an example, doesn't need full documentation
• document stopwords fully on analysis/analyzers/stop-analyzer.asciidoc (note the default here is still english)
• refer to the above from standard and pattern analyzers, noting that the default for these is _none_

No need to close this PR, you can just update it and push again.

thanks for helping @mrkamel

[clintongormley]

also please include the coming[1.0.0.RC1,Previously defaulted to the English stopwords] on the pattern analyzer page, and please rebase your PR on master.

ta

[mrkamel]

Before, may i finally ask why ES actually uses such kind of special keyword _none_? I'd at least expect stopwords: [] to work while being much more self-explanatory and yaml-like, thus making this PR unnecessary. However, stopwords: [] or stopwords: "" doesn't work in my tests.

Sorry for being pedantic, but i don't easily find an obvious hint within the sources.

[clintongormley]

Good question, and I don't know either :) Looks like an empty list isn't being considered currently.

I've opened #4709 to get support added (but this may take a little while as we're getting ready for the next release now).

[clintongormley]

Merged! Many thanks for the PR @mrkamel