Skip to content

Add ingest CRUD APIs#14047

Merged
martijnvg merged 1 commit intoelastic:feature/ingestfrom
martijnvg:ingest/add_crud_apis
Oct 9, 2015
Merged

Add ingest CRUD APIs#14047
martijnvg merged 1 commit intoelastic:feature/ingestfrom
martijnvg:ingest/add_crud_apis

Conversation

@martijnvg
Copy link
Member

Added first version of put, get and delete pipeline APIs.

In order to support deletes the PipelineStore needs to also check if all the pipelines it has loaded do still exist.

The documentation is minimal and sits next to all the docs of all other plugins. We may want to move the ingest docs to the plugin itself?

@martijnvg martijnvg added review :Distributed/Ingest Node Execution or management of Ingest Pipelines labels Oct 9, 2015
@martijnvg martijnvg force-pushed the ingest/add_crud_apis branch from 263a3cb to 06ee38f Compare October 9, 2015 13:09
@jpountz jpountz mentioned this pull request Oct 9, 2015
@martijnvg martijnvg force-pushed the ingest/add_crud_apis branch from 06ee38f to 5a3c75e Compare October 9, 2015 16:29
@martijnvg martijnvg merged commit 5a3c75e into elastic:feature/ingest Oct 9, 2015
dnhatn added a commit that referenced this pull request Jan 24, 2026
The competitive benchmark shows a significant performance regression 
from #140475. This is likely due to the high number of CPUs in the 
benchmark, which results in many small slices and adds significant
overhead.

It is better to revert this change for now and adjust the partitioning 
for cases with high CPU and more than one shard before merging again.

Relates #14047
dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Jan 24, 2026
The competitive benchmark shows a significant performance regression
from elastic#140475. This is likely due to the high number of CPUs in the
benchmark, which results in many small slices and adds significant
overhead.

It is better to revert this change for now and adjust the partitioning
for cases with high CPU and more than one shard before merging again.

Relates elastic#14047

(cherry picked from commit 1146b88)

# Conflicts:
#	x-pack/plugin/esql/qa/testFixtures/src/main/resources/k8s-timeseries.csv-spec
elasticsearchmachine pushed a commit that referenced this pull request Jan 24, 2026
The competitive benchmark shows a significant performance regression
from #140475. This is likely due to the high number of CPUs in the
benchmark, which results in many small slices and adds significant
overhead.

It is better to revert this change for now and adjust the partitioning
for cases with high CPU and more than one shard before merging again.

Relates #14047

(cherry picked from commit 1146b88)

# Conflicts:
#	x-pack/plugin/esql/qa/testFixtures/src/main/resources/k8s-timeseries.csv-spec
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Ingest Node Execution or management of Ingest Pipelines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant