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

Deprecate types in simulate pipeline requests. #37731

Closed
jtibshirani opened this issue Jan 22, 2019 · 2 comments
Closed

Deprecate types in simulate pipeline requests. #37731

jtibshirani opened this issue Jan 22, 2019 · 2 comments
Assignees
Labels
:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >deprecation

Comments

@jtibshirani
Copy link
Contributor

The _type field can currently be provided in the docs section of simulate pipeline request (as in the example https://www.elastic.co/guide/en/elasticsearch/reference/master/simulate-pipeline-api.html). It would be great to deprecate the use of _type in these requests, making sure to deprecate the relevant methods in the high-level REST response as well.

This API is likely less urgent to address than the others we've encountered, because a typeless request option is already available. In particular, it is already possible to omit the _type field in requests (it looks like a default of _type will be used if a type is not provided).

@jtibshirani jtibshirani added >deprecation :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP labels Jan 22, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

@jtibshirani
Copy link
Contributor Author

Just a note that 'team types removal' was hoping the core features team could take this on. Let us know if this seems off or if you have any questions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >deprecation
Projects
None yet
Development

No branches or pull requests

3 participants