Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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