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 Repository Stats API #62297

Merged
merged 1 commit into from
Sep 14, 2020

Conversation

tlrx
Copy link
Member

@tlrx tlrx commented Sep 14, 2020

This pull request deprecates the Repository Stats API added in 7.8.0 as an experimental API behind a feature flag. The goal is to deprecate this API in 7.10.0 and remove it in a follow up PR in 8.0.0.

This API is now superseded by the Repositories Metering API added in 7.10.0.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Snapshot/Restore)

@elasticmachine elasticmachine added the Team:Distributed Meta label for distributed team (obsolete) label Sep 14, 2020
@original-brownbear
Copy link
Member

@tlrx I'm all for this, but I wonder, did we ever document this API in a way that means we can't just remove it outright?

@tlrx
Copy link
Member Author

tlrx commented Sep 14, 2020

There was a bit of documentation (which has been removed in #62217). This is experimental API hidden behind a feature flag of a not yet release feature so I was also wondering how to remove this.

But we have processes to remove APIs... so I'm just following them in case something somewhere is using this (which I don't expect).

Copy link
Member

@original-brownbear original-brownbear left a comment

Choose a reason for hiding this comment

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

LGTM, lets do it the official way :)

@tlrx tlrx merged commit a25f06d into elastic:master Sep 14, 2020
@tlrx tlrx deleted the deprecate-repository-stats-api branch September 14, 2020 11:44
@tlrx
Copy link
Member Author

tlrx commented Sep 14, 2020

Thanks Armin!

tlrx added a commit that referenced this pull request Sep 14, 2020
This commit deprecates the Repository Stats API added in 7.8.0 as
an experimental API behind a feature flag. The goal is to deprecate
this API in 7.10.0 and remove it in a follow up PR in 8.0.0.

This API is now superseded by the Repositories Metering API.
tlrx added a commit that referenced this pull request Sep 24, 2020
Now the Repository Stats API is deprecated in 7.10.0 (#62297) 
we can remove it in 8.0.0.

I'm labeling this as >breaking to respect the development 
process but we're talking about an experimental API that 
was never released (behind a feature flag).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>deprecation :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed Meta label for distributed team (obsolete) v7.10.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants