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

sql: add indexes (#89988) #1442

Merged
fpeters merged 1 commits from wip/89988-postgresql-more-indexes into main 2024-04-29 16:28:22 +02:00
Conversation 1 Commits 1 Files Changed 2 +19 -6
fpeters commented 2024-04-26 13:28:25 +02:00
Owner
Copy Link

@pducroquet :

  • index gin sur actions_roles_array de wcs_all_forms en le conditionnant à la visibilité éventuellement (status <> 'draft' and anonymised is null and not is_at_endpoint)

Il y a déjà un USING gin(actions_roles_array) WHERE (anonymised IS NULL AND is_at_endpoint = false) il n'est pas suffisant, ou à revoir ?

@pducroquet : > - index gin sur actions_roles_array de wcs_all_forms en le conditionnant à la visibilité éventuellement (status <> 'draft' and anonymised is null and not is_at_endpoint) Il y a déjà un `USING gin(actions_roles_array) WHERE (anonymised IS NULL AND is_at_endpoint = false)` il n'est pas suffisant, ou à revoir ?
fpeters added 1 commit 2024-04-26 13:28:26 +02:00
gitea/wcs/pipeline/head This commit looks good Details
6cfbdf99f0 sql: add indexes (#89988)
fpeters changed title from WIP: sql: add indexes (#89988) to sql: add indexes (#89988) 2024-04-26 13:52:55 +02:00
fpeters requested review from pducroquet 2024-04-26 13:53:05 +02:00
pducroquet approved these changes 2024-04-29 00:25:13 +02:00
pducroquet commented 2024-04-29 00:25:48 +02:00
Owner
Copy Link

@pducroquet :

  • index gin sur actions_roles_array de wcs_all_forms en le conditionnant à la visibilité éventuellement (status <> 'draft' and anonymised is null and not is_at_endpoint)

Il y a déjà un USING gin(actions_roles_array) WHERE (anonymised IS NULL AND is_at_endpoint = false) il n'est pas suffisant, ou à revoir ?

Erreur de ma part en lisant le schéma sur toulouse, l'index existant est bien suffisant.

> @pducroquet : > > > - index gin sur actions_roles_array de wcs_all_forms en le conditionnant à la visibilité éventuellement (status <> 'draft' and anonymised is null and not is_at_endpoint) > > Il y a déjà un `USING gin(actions_roles_array) WHERE (anonymised IS NULL AND is_at_endpoint = false)` il n'est pas suffisant, ou à revoir ? Erreur de ma part en lisant le schéma sur toulouse, l'index existant est bien suffisant.
fpeters force-pushed wip/89988-postgresql-more-indexes from 6cfbdf99f0 to e3d936be32 2024-04-29 09:29:28 +02:00 Compare
fpeters force-pushed wip/89988-postgresql-more-indexes from e3d936be32 to 27125f9830 2024-04-29 16:21:23 +02:00 Compare
fpeters reviewed 2024-04-29 16:28:10 +02:00
wcs/backoffice/filter_fields.py
@ -423,0 +427,4 @@
]
)
],
order_by='ascii_name',
fpeters commented 2024-04-29 16:28:10 +02:00
Author
Owner
Copy Link

Pour info j'ai du ajouter ça pour être sûr qu'un test donne toujours les mêmes résultats. (il y a juste l'order_by ajouté, c'est juste que ça allongeait trop la ligne).

Pour info j'ai du ajouter ça pour être sûr qu'un test donne toujours les mêmes résultats. (il y a juste l'order_by ajouté, c'est juste que ça allongeait trop la ligne).
fpeters merged commit 27125f9830 into main 2024-04-29 16:28:22 +02:00
fpeters deleted branch wip/89988-postgresql-more-indexes 2024-04-29 16:28:22 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pducroquet
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#1442
No description provided.
Delete Branch "wip/89988-postgresql-more-indexes"

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?