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

fix(db): set correct dbms short name postgresql #3364

Merged
merged 1 commit into from
Nov 25, 2024
Merged

Conversation

silvestre
Copy link
Member

@silvestre silvestre commented Nov 25, 2024

Issue

The index index_app_metrics was not created on PostgreSQL leading to bad performance.

Fix

The correct DBMS/shortname for PostgreSQL is postgresql as documented

# Issue

The index `index_app_metrics` was not created on PostgreSQL leading to bad performance.

# Fix

The correct `DBMS/shortname` for PosgreSQL is  `postgresql` as [documented](https://docs.liquibase.com/start/tutorials/home.html)
Copy link

sonarcloud bot commented Nov 25, 2024

Copy link
Contributor

@bonzofenix bonzofenix left a comment

Choose a reason for hiding this comment

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

lgtm

@silvestre silvestre merged commit 2755c1e into main Nov 25, 2024
31 checks passed
@silvestre silvestre deleted the add-app_metric-index branch November 25, 2024 12:51
@silvestre silvestre added the bug label Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants