Skip to content

Include version in GetPipeline response #6082

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

Closed
mpwoz opened this issue Jan 28, 2022 · 1 comment
Closed

Include version in GetPipeline response #6082

mpwoz opened this issue Jan 28, 2022 · 1 comment

Comments

@mpwoz
Copy link

mpwoz commented Jan 28, 2022

Is your feature request related to a problem? Please describe.
GetPipeline doesn't include pipeline version

Describe the solution you'd like
The ES ingest/get_pipeline API includes a version field in the response: https://www.elastic.co/guide/en/elasticsearch/reference/current/get-pipeline-api.html
This is needed for making concurrency-safe updates to an existing pipeline (e.g. adding a processor)

The Nest client does not include this field in the response. https://github.com/elastic/elasticsearch-net/blob/master/src/Nest/Ingest/GetPipeline/GetPipelineResponse.cs

Describe alternatives you've considered
Right now, I'm considering falling back to the low-level REST Api, but would prefer to use the high level nest API as it already includes everything I need (except version) https://github.com/elastic/elasticsearch-net#falling-back-to-elasticsearchnet

Additional context

@stevejgordon
Copy link
Contributor

Thanks for catching and raising this @mpwoz. I've fixed this in #6085 which will go out in the next release.

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

No branches or pull requests

2 participants