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

refactor (graphql-server): Improves build script #18821

Merged

Conversation

gustavotrott
Copy link
Collaborator

This PR introduces 3 tweaks:

Command drop database

  • Adds the param with (force) in order to avoid the error database "bbb_graphql" is being accessed by other users

image

https://www.postgresql.org/docs/current/sql-dropdatabase.html
It requires PostgreSQL >= 13 (by default BBB installs v14.9)

Execution of bbb_schema.sql

  • Remove the param -a to reduce logs

image

  • Include the param -q to reduce logs

image

https://www.postgresql.org/docs/current/app-psql.html

@gustavotrott gustavotrott added this to the Release 3.0 milestone Sep 21, 2023
@TiagoJacobs TiagoJacobs merged commit be8f224 into bigbluebutton:develop Sep 21, 2023
15 checks passed
@sonarcloud
Copy link

sonarcloud bot commented Sep 21, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-actions
Copy link

Automated tests Summary

All the CI tests have passed!

@gustavotrott gustavotrott deleted the graphql-server-db-tweaks branch October 11, 2023 17:58
@danimo danimo mentioned this pull request Nov 28, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants