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

[skip ci] docs: dashboard import/export API #22835

Merged
merged 2 commits into from
Sep 7, 2018

Conversation

epixa
Copy link
Contributor

@epixa epixa commented Sep 7, 2018

Closes #14872

@epixa
Copy link
Contributor Author

epixa commented Sep 7, 2018

/cc @AlonaNadler @jimgoodwin

--------------------------------------------------
// KIBANA

A successful call returns a response code of `200`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This API returns 200 regardless if there are errors within the bulk operations or not. That indicates that the bulk operation itself was successful, but the user will still need to check the results for any errors that may occurred.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, good catch.

Copy link
Member

@simianhacker simianhacker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just left a short note that we might want to add. I know in the past people were confused that by the fact that it returns 200 on import even when individual operations fail but it's designed this way to be consistent with how the bulk api's work in Elasticsearch.

@epixa
Copy link
Contributor Author

epixa commented Sep 7, 2018

I pushed some clarifying text for the error handling on import.

@epixa epixa merged commit 54ab5a7 into elastic:master Sep 7, 2018
@epixa epixa deleted the doc-dashboardimport branch September 7, 2018 19:52
epixa added a commit that referenced this pull request Sep 7, 2018
* docs: dashboard import/export API

* docs: clarify response handling of errors in import api
epixa added a commit that referenced this pull request Sep 7, 2018
* docs: dashboard import/export API

* docs: clarify response handling of errors in import api
@epixa
Copy link
Contributor Author

epixa commented Sep 7, 2018

6.x/6.5 57b6aaf
6.4 e345d26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants