Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: Document special case for type parameter requirement for _mget #6670

Closed
ppf2 opened this issue Jul 1, 2014 · 3 comments
Closed

Docs: Document special case for type parameter requirement for _mget #6670

ppf2 opened this issue Jul 1, 2014 · 3 comments
Assignees
Labels

Comments

@ppf2
Copy link
Member

ppf2 commented Jul 1, 2014

The current documentation (http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/docs-multi-get.html#docs-multi-get) indicates that type is optional in the _mget request.

Per #6633, in the case of multiple types sharing the same ID, in order for _mget to fetch back documents sharing the same ID across types, a _type+_id parameter have to be specified (i.e. cannot simply specify _id to fetch documents across types sharing the same id).

GET /sameid/_mget/
{
  "docs" : [
        {
            "_type":"typeA",
            "_id" : "1"
        },
        {
            "_type":"typeB",
            "_id" : "1"
        }
    ]
}

Would be nice to add this as a special case in the documentation for _mget.

@ppf2 ppf2 added the doc label Jul 1, 2014
@dadoonet
Copy link
Member

dadoonet commented Jul 2, 2014

@clintongormley
Copy link

@ppf2 want to send a PR?

dadoonet added a commit that referenced this issue Jul 3, 2014
@clintongormley
Copy link

@dadoonet your commit breaks the docs build because you're using an ID which is not unique across the docs, ie [[type]]

Please fix

@clintongormley clintongormley reopened this Jul 3, 2014
dadoonet added a commit that referenced this issue Jul 3, 2014
Change ID to `[[mget-type]]`

Closes #6670.

(cherry picked from commit 162c62d)
@clintongormley clintongormley changed the title Discuss briefly the special case for type parameter requirement for _mget Docs: Document special case for type parameter requirement for _mget Jul 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants