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.

Sign up for GitHub

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

Jump to bottom

V14: Modifications de la datagrid non prises en compte pour les gros fichiers #2200

Closed
eonm-pro opened this issue Oct 30, 2024 · 4 comments · Fixed by #2396
Closed

V14: Modifications de la datagrid non prises en compte pour les gros fichiers #2200

eonm-pro opened this issue Oct 30, 2024 · 4 comments · Fixed by #2396
Assignees
@eonm-pro @yannguillemin @jonathanarnault
Labels
🐛 Major defect Le problème limite le fonctionnement du processus standard

Comments

@eonm-pro
Copy link
Collaborator

Les modifications des valeurs de la datagrid Lodex (backoffice) ne sont pas prise en compte pour les gros fichiers (456 231 lignes/724 Mo).

En inspectant la console de développement on obtient l'erreur suivante :

disctinct too big, 16mb cap

La notification consécutive à la sauvegarde du résultat affiche pourtant un succès.

image

Reproduire

  1. Depuis la datagrid Lodex modifier la valeur d'une cellule et sauvegarder la modification
  2. Recharger ou changer de page dans la datagrid
  3. La modification de valeur n'a pas été prise en compte

Environnement :

  • Google Chrome Version 130.0.6723.70 (Build officiel) (64 bits)
  • également testé sur Firefox Nightly (134.0a1)
  • cnrsinist/lodex:14.0.63
@eonm-pro eonm-pro added the Triage Nouveau problème ou besoin à étudier et à classer label Oct 30, 2024
@touv touv added 🐛 Major defect Le problème limite le fonctionnement du processus standard and removed Triage Nouveau problème ou besoin à étudier et à classer labels Nov 5, 2024
@Alexmarmelab
Copy link
Collaborator

@eonm-pro merci d'avoir remonté la carte.

Est-ce que tu pourrais ajouter le lien de l'instance dans laquelle t'as rencontré le problème ?

@eonm-pro
Copy link
Collaborator Author

@eonm-pro merci d'avoir remonté la carte.

Est-ce que tu pourrais ajouter le lien de l'instance dans laquelle t'as rencontré le problème ?

J'ai créé une instance sur le DEV : test-revue-sommaire

@jonathanarnault jonathanarnault self-assigned this Jan 22, 2025
@jonathanarnault jonathanarnault mentioned this issue Jan 22, 2025
Merged
@jonathanarnault jonathanarnault changed the title Modifications de la datagrid non prises en compte pour les gros fichiers (V14) Modifications de la datagrid non prises en compte pour les gros fichiers Jan 23, 2025
@jonathanarnault jonathanarnault changed the title (V14) Modifications de la datagrid non prises en compte pour les gros fichiers V14: Modifications de la datagrid non prises en compte pour les gros fichiers Jan 23, 2025
@Alexmarmelab
Copy link
Collaborator

Vu au daily : ok pour Mathis et Yann
Mathis a demandé à d'autres personnes de tester

@eonm-pro
Copy link
Collaborator Author

Vu et testé par @camilledesalabert. C'est OK pour elle également.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@eonm-pro eonm-pro

@yannguillemin yannguillemin

@jonathanarnault jonathanarnault

Labels
🐛 Major defect Le problème limite le fonctionnement du processus standard
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix(dataset): Use aggregate instead of distinct to count unique fields to avoid oom Inist-CNRS/lodex
5 participants
@touv @eonm-pro @yannguillemin @jonathanarnault @Alexmarmelab