[checkmk-commits] [tribe29/checkmk] 91d866: Let PackedConfigStore.write take a mapping

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 91d866dc3898a3e76afa5fc39e93f8bb2a3fe890 https://github.com/tribe29/checkmk/commit/91d866dc3898a3e76afa5fc39e93f8bb2… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-12-23 (Wed, 23 Dec 2020) Changed paths: M cmk/base/config.py M tests/unit/cmk/base/test_config.py Log Message: ----------- Let PackedConfigStore.write take a mapping `PackedConfigStore.read()` returns a mapping so that taking a mapping on `write()` makes the class internally consistent. This also makes it clear that there cannot be two keys with the same name. CMK-6846 Change-Id: I417d8525fd62ae1f1dade77d7b66ed849c6ad4aa Commit: c94f35cb1a7b7c8d1e4f7291fab97f3edaf12f8b https://github.com/tribe29/checkmk/commit/c94f35cb1a7b7c8d1e4f7291fab97f3ed… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-12-23 (Wed, 23 Dec 2020) Changed paths: M cmk/base/config.py M tests/unit/cmk/base/test_config.py M tests/unit/cmk/base/test_core_config.py Log Message: ----------- Serialize the packed config with pickle The data is now serialized only once using pickle and the clear text version is not saved anymore. Replacing the double serialization (to string as well as repr/eval/marshal) represents a factor 35+ speed improvement on a small test set. We expect this factor to scale well to larger test sets as well. CMK-6846 Change-Id: Icbeceadb3da63e58f10130dd4cb03c844d90a7a9 Compare: https://github.com/tribe29/checkmk/compare/4effbb3b43cf...c94f35cb1a7b