entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 45 Releases Activity

WIP: sql: send cron in application_name to show origin of queries in PostgreSQL (#75795) #200

Closed
pducroquet wants to merge 1 commits from wip/75795-sql-application_name into main
pull from: wip/75795-sql-application_name
merge into: entrouvert:main
Conversation 1 Commits 1 Files Changed 2 +2 -1
pducroquet commented 2023-03-27 08:41:28 +02:00
Owner
Copy Link
No description provided.
pducroquet added 1 commit 2023-03-27 08:41:28 +02:00
fpeters requested changes 2023-03-27 10:20:45 +02:00
fpeters left a comment
Owner
Copy Link

J'ai un peu peur d'une situation où la config serait enregistrée lors du cron (type parce que migration qui tourne sur les champs et qui toucherait ceux du profil utilisateur) et qu'ensuite elle se trouve donc utilisée partout; je serais plutôt pour quelque chose type get_publisher().sql_application_name = 'cron'; puis dans le get_connection exploiter ça.

J'ai un peu peur d'une situation où la config serait enregistrée lors du cron (type parce que migration qui tourne sur les champs et qui toucherait ceux du profil utilisateur) et qu'ensuite elle se trouve donc utilisée partout; je serais plutôt pour quelque chose type `get_publisher().sql_application_name = 'cron'`; puis dans le get_connection exploiter ça.
fpeters commented 2023-07-18 14:10:05 +02:00
Owner
Copy Link

Nouvelle PR à venir.

Nouvelle PR à venir.
fpeters closed this pull request 2023-07-18 14:10:05 +02:00
All checks were successful
gitea/wcs/pipeline/head This commit looks good
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#200
No description provided.
Delete Branch "wip/75795-sql-application_name"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?