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

MSSQL/Postgres: List views in table dropdown as well #62867

Merged
merged 2 commits into from Feb 6, 2023

Conversation

zoltanbedi
Copy link
Member

@zoltanbedi zoltanbedi commented Feb 3, 2023

This PR fixes an issue that the table dropdown for the MSSQL and Postgres datasource only showed tables and not views. This pr fixes it by removing the constraint in the table query.

Which issue(s) does this PR fix?:

Fixes #59775 #62674

Special notes for your reviewer:

@zoltanbedi zoltanbedi added type/bug add to changelog backport v9.4.x Mark PR for automatic backport to v9.4.x labels Feb 3, 2023
@zoltanbedi zoltanbedi added this to the 9.4.0 milestone Feb 3, 2023
@zoltanbedi zoltanbedi self-assigned this Feb 3, 2023
@zoltanbedi zoltanbedi requested review from a team, codeincarnate and mdvictor and removed request for a team February 3, 2023 13:07
@zoltanbedi zoltanbedi added no-backport Skip backport of PR and removed type/bug backport v9.4.x Mark PR for automatic backport to v9.4.x labels Feb 3, 2023
@zoltanbedi zoltanbedi modified the milestones: 9.4.0, 9.5.0 Feb 3, 2023
Copy link
Contributor

@baldm0mma baldm0mma left a comment

Choose a reason for hiding this comment

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

lgtm!

@zoltanbedi zoltanbedi merged commit d103c58 into main Feb 6, 2023
@zoltanbedi zoltanbedi deleted the zoltan/sql-query-views branch February 6, 2023 14:16
ryantxu pushed a commit that referenced this pull request Feb 8, 2023
* MSSQL: Query views for table dropdown

* Postgres: Query views for table dropdown
ryantxu pushed a commit that referenced this pull request Mar 2, 2023
* MSSQL: Query views for table dropdown

* Postgres: Query views for table dropdown
@Helix751
Copy link

Helix751 commented May 22, 2023

Is this issue fixed? Grafana 9.4.3 is not showing database Views, just tables in the core installed MS SQL Server plugin.

Edit: This seems to be fixed since Milestone 9.5.0. I upgraded my Grafana installation to current 9.5.2 and Views now pop up like tables are, as expected. Thanks!

@baldm0mma
Copy link
Contributor

@Helix751 Good to hear! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show database views in SQL query builder for MS SQL
4 participants