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

MM-59410 - Adding dim_server_info to the "Feature usage" explore #1020

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

catalintomai
Copy link
Contributor

@catalintomai catalintomai commented Jul 11, 2024

Summary

Adding join to the dim_server_info view to the "Feature Usage" explore. The reason is to help implement MM-59410, to use the "hosting type" to be able to display either cloud or self-hosted customer info, depending on the server type. Technically, there should be no need for that right now, the cloud installs have the full set of features, but could not find any other explore that would make sense to use in the context of MM-59410.

Other options would be to create a separate explore for MM-59410 and alike or to use smth. like rpt_active_user_base (which coalesces cloud and self-hosted customer metadata and is the only explore that uses the latest customer info - dim_latest_server_customer_info).

Ticket Link

https://mattermost.atlassian.net/browse/MM-59410

@ifoukarakis
Copy link
Contributor

Adding a sanity check for this: mattermost/mattermost-data-warehouse#1559. Let's resolve any issues and we can follow up on this PR.

@catalintomai catalintomai added the Do Not Merge Should not be merged until this label is removed label Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge Should not be merged until this label is removed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants