-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Feat db pools eldad #4330
Feat db pools eldad #4330
Conversation
…into feat-db-pools-eldad
…at-db-pools-eldad
…at-db-pools-eldad
} catch (\Throwable $th) { | ||
Console::error('Queue............disconnected 👎'); | ||
$configs = [ | ||
'Console.DB' => Config::getParam('pools-console'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any reason to use the dot in the key names? We haven't used this convention before
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I felt a space would be weird here because of the format, and camelCase would indicate a code concept while this should be a human readable format 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
being an array key I think its valid to use console-db
and projects-db
or use the same key as the config
pools-console
and pools-projects
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Just some last comments
What does this PR do?
Updated for the database pools PR.
App::getEnv()
App::isDevelopment()
Pools::getDatabase()
to use generic DSN connections_APP_DB__APP_DB_CONSOLEROOT_PASS
to be_APP_DB_CONSOLE
_APP_DB_CONSOLE
and_APP_DB_PROJECT
from docsOpen Questions:
Test Plan
Use existing tests
Related PRs and Issues
#3423
Have you read the Contributing Guidelines on issues?
Yes