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

Add "deprecated" boolean to metadata.yaml #1827

Closed
jklukas opened this issue Feb 18, 2021 · 1 comment
Closed

Add "deprecated" boolean to metadata.yaml #1827

jklukas opened this issue Feb 18, 2021 · 1 comment

Comments

@jklukas
Copy link
Contributor

jklukas commented Feb 18, 2021

As we move towards publishing documentation about views into derived data, it could be very helpful to be able to annotate some views as deprecated. For example, we have a general policy of having user-facing views be unversioned, but we have many views that are duplicated to versioned and unversioned names for historical reasons. It would be great to be able to hide the views that we consider deprecated, or at least deemphasize them in our generated documentation.

We often need to keep deprecated views around so as to not break existing queries, but I'd like to see us lessen the impact of an ever-growing number of views for users to sort through.

@ANich
Copy link
Contributor

ANich commented Dec 15, 2023

This was implemented in #4213. We discussed this briefly during a recent infra meeting and as I understand it the semantics underlying the deprecation flag were built with tables in mind but should apply to views as well, so I think it's fair to close this. I'll likely be testing this soon with telemetry.clients_daily_v6 #4660.

cc fysa @alekhyamoz @whd

@ANich ANich closed this as completed Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants