Project

General

Profile

Development #65497

moins attendre pour préférer un snapshot complet à un patch

Added by Frédéric Péters about 1 month ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
20 May 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

via #65485,

Si je prends les deux plus récentes modifications, je télécharge, et je diff,

--- snapshot-workflow-21301-20220424-1811.wcs   2022-05-20 09:33:15.837723115 +0200
+++ snapshot-workflow-21302-20220424-1812.wcs   2022-05-20 09:33:09.497675588 +0200
@@ -209977,7 +209977,7 @@
                 <prefill>
                   <locked>False</locked>
                   <type>string</type>
-                  <value>{{mairie_creation_8_var_mairie|default:""}}</value>
+                  <value>{% if form_name == "Demande de livret de famille suite &#224; erreur" %}{{form_var_remise_lf_orleans}}{% else %}{{mairie_creation_8_var_mairie|default:""}}{% endif %}</value>
                 </prefill>
                 <items>
                   <item>Mairie centrale</item>

mais dans la table il y a un diff qui est bien plus volumineux.
(...)
Ce qui est logique le diff est fait par rapport à la dernière version complète.

            if len(obj.serialization) > len(patch):
                # serialization is bigger than patch, store patch

j'étais juste avant sur l'idée de faire un len(obj.serialization) / 10 et chercher ce facteur mais peut-être que c'est aussi à combiner avec un truc statique genre len(patch) > 1_000_000.

Voyons ça ici.


Files


Related issues

Related to w.c.s. - Development #65485: Table snapshot très volumineuse (-> partitionner la table)Solution déployée20 May 2022

Actions

Associated revisions

Revision 36c34a72 (diff)
Added by Frédéric Péters about 1 month ago

snapshots: lower size where a full snapshot is preferred over a diff (#65497)

History

#1

Updated by Frédéric Péters about 1 month ago

  • Related to Development #65485: Table snapshot très volumineuse (-> partitionner la table) added
#2

Updated by Frédéric Péters about 1 month ago

#3

Updated by Lauréline Guerin about 1 month ago

  • Status changed from Solution proposée to Solution validée
#4

Updated by Frédéric Péters about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 36c34a72fb2ee2119c5b198dcd074cd068e23fb3
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri May 20 10:29:22 2022 +0200

    snapshots: lower size where a full snapshot is preferred over a diff (#65497)
#5

Updated by Transition automatique about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF