Skip to content

Convert index path listeners to single path#72511

Merged
rjernst merged 2 commits intoelastic:masterfrom
rjernst:mdp21
Apr 29, 2021
Merged

Convert index path listeners to single path#72511
rjernst merged 2 commits intoelastic:masterfrom
rjernst:mdp21

Conversation

@rjernst
Copy link
Copy Markdown
Member

@rjernst rjernst commented Apr 29, 2021

The listeners for deletion of index and shard paths previously accepted
an array of paths, since they could exist on several different data
paths. This commit converts the consumer to take a single path.

relates #71205

The listeners for deletion of index and shard paths previously accepted
an array of paths, since they could exist on several different data
paths. This commit converts the consumer to take a single path.

relates elastic#71205
@rjernst rjernst added :Core/Infra/Core Core issues without another label >refactoring v8.0.0 labels Apr 29, 2021
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Apr 29, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Copy link
Copy Markdown
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

@rjernst rjernst merged commit fd138f5 into elastic:master Apr 29, 2021
@rjernst rjernst deleted the mdp21 branch April 29, 2021 21:39
@rjernst rjernst mentioned this pull request Sep 30, 2021
17 tasks
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Oct 11, 2021
This reverts commit fd138f5.

The revert was effectively conflict free, there was only a very minor
static import that needed fixing.

relates elastic#78525
relates elastic#71205
rjernst added a commit that referenced this pull request Oct 11, 2021
This reverts commit fd138f5.

The revert was effectively conflict free, there was only a very minor
static import that needed fixing.

relates #78525
relates #71205
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >refactoring Team:Core/Infra Meta label for core/infra team v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants