Indices.All vs Indices.AllIndicies

[niemyjski]

There's no docs on this that I can see. Any reason why there is both?

    public static Indices All { get; } = new Indices(new Indices.AllIndicesMarker());

    public static Indices AllIndices { get; } = new Indices(new Indices.AllIndicesMarker());

[Mpdreamz]

Sharp eye :)

It's for when you statically import Indices and Types and All becomes ambiguous.

using static Nest.Indices;

The preferred static import target though is

using static Nest.Infer;

for everything (Fields, Type(s), Index, Indices and Id(T))

We'll be more conservative how we expose this in 5.0 and write proper documentation around this (beyond the inferrence pages we already have https://www.elastic.co/guide/en/elasticsearch/client/net-api/current/field-inference.html#nest-infer)

[niemyjski]

Thanks :). Could we use an import alias?

[niemyjski]

I just hit this again when trying to upgrade from 1.x to 5.x

[niemyjski]

There are no docs on this.

[russcam]

Did you see the indices paths documentation?

We're going to spend some time to go over the documentation to include elements that are not covered currently and to arrange the content in a more coherent fashion.

[niemyjski]

I did not, I always use up-to-date source as my docs most of the time. With this said I have a correction to my statement: There are no xml docs for this :)

Why not remove All and have AllIndicies then it would remove all confusion?