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

reprise de la création d'index (#85108) #1032

Merged
fpeters merged 6 commits from wip/85108-indexes into main 2024-01-26 08:45:39 +01:00
Conversation 0 Commits 6 Files Changed 1 +105 -80
fpeters commented 2024-01-16 14:31:22 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/85108-indexes from 7db445285a to f991af7d5c 2024-01-16 14:44:19 +01:00 Compare
fpeters force-pushed wip/85108-indexes from f991af7d5c to a96b0dd5f6 2024-01-16 15:14:19 +01:00 Compare
fpeters force-pushed wip/85108-indexes from a96b0dd5f6 to e7b75e6afa 2024-01-16 15:50:56 +01:00 Compare
fpeters reviewed 2024-01-16 16:13:32 +01:00
wcs/sql.py
@ -1542,3 +1512,1 @@
'''CREATE INDEX IF NOT EXISTS %s_fts ON %s USING gin(fts)''' % ('wcs_all_forms', 'wcs_all_forms')
)
create_index = 'CREATE INDEX IF NOT EXISTS'
fpeters commented 2024-01-16 16:10:50 +01:00
Author
Owner
Copy Link

Dans do_global_views le CONCURRENTLY ne marche pas parce que ça se trouve dans une transaction et psycopg2.errors.ActiveSqlTransaction: ERREUR: CREATE INDEX CONCURRENTLY ne peut pas être exécuté dans un bloc de transaction.

On fait sans.

Dans do_global_views le CONCURRENTLY ne marche pas parce que ça se trouve dans une transaction et `psycopg2.errors.ActiveSqlTransaction: ERREUR: CREATE INDEX CONCURRENTLY ne peut pas être exécuté dans un bloc de transaction`. On fait sans.
wcs/sql.py
@ -1683,0 +1650,4 @@
_sql_indexes = None
@classmethod
def do_indexes(cls, cur, concurrently=False):
fpeters commented 2024-01-16 16:13:01 +01:00
Author
Owner
Copy Link

C'est factorisé au niveau de la classe, et on passe à un système déclaratif.

(un autre jour les do_xxx_table seront aussi déplacés/uniformisés en classmethod).

C'est factorisé au niveau de la classe, et on passe à un système déclaratif. (un autre jour les do_xxx_table seront aussi déplacés/uniformisés en classmethod).
wcs/sql.py
@ -2843,0 +2826,4 @@
'users_name_identifiers_idx ON users USING gin(name_identifiers)',
'users_fts ON users USING gin(fts)',
'users_roles_idx ON users USING gin(roles)',
]
fpeters commented 2024-01-16 16:08:56 +01:00
Author
Owner
Copy Link

On est sur un système déclaratif générique réduit au plus simple.

On est sur un système déclaratif générique réduit au plus simple.
wcs/sql.py
@ -5279,2 +5286,2 @@
for formdef in FormDef.select() + CardDef.select():
do_formdef_indexes(formdef, created=False, conn=conn, cur=cur)
# 99: add more indexes
set_reindex('sqlindexes', 'needed', conn=conn, cur=cur)
fpeters commented 2024-01-16 16:11:22 +01:00
Author
Owner
Copy Link

On pose désormais un marqueur, pour exécution après coup.

On pose désormais un marqueur, pour exécution après coup.
fpeters changed title from WIP: reprise de la création d'index (#85108) to reprise de la création d'index (#85108) 2024-01-16 16:13:51 +01:00
pducroquet approved these changes 2024-01-19 11:49:15 +01:00
fpeters merged commit 8da0f623ba into main 2024-01-26 08:45:39 +01:00
fpeters deleted branch wip/85108-indexes 2024-01-26 08:45:39 +01: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#1032
No description provided.
Delete Branch "wip/85108-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?