idty_by_username

Add IdtyByUsername GVA query.

J'ai dû ajouter la pubkey dans gva_identities pour le revert uniquement.

@librelois

db/src/lib.rs

@@ -41,8 +41,8 @@ pub(crate) use duniter_core::dbs::kv_typed::db_schema;
 pub(crate) use duniter_core::dbs::kv_typed::prelude::*;
 pub(crate) use duniter_core::dbs::smallvec::SmallVec;
 pub(crate) use duniter_core::dbs::{
-    bincode_db, CorruptedBytes, HashKeyV2, PubKeyKeyV2, SourceAmountValV2, ToDumpString,
-    WalletConditionsV2,
+    bincode_db, CorruptedBytes, HashKeyV2, PubKeyKeyV2, PubKeyValV2, SourceAmountValV2,
+    ToDumpString, WalletConditionsV2,
 };
 pub(crate) use duniter_core::wallet::prelude::*;
 pub(crate) use duniter_core::wot::WotId;
@@ -67,6 +67,7 @@ db_schema!(
             GvaUtxoIdDbV1,
             SourceAmountValV2
         ],
+        ["idty_by_username", IdtyByUsername, String, PubKeyValV2],
         [
             "scripts_by_pubkey",
             ScriptsByPubkey,

[Éloïs]

Une optimisation possible: Remplacer String par ArrayString<100>

[Pascal Engélibert]

ArrayString n'impl pas les bons traits, donc il faudrait faire un wrapper... et il a une taille constante, veut-on vraiment prendre 100 octets par username quand ils font en moyenne moins de 10 ?

Si on veut optimiser ça, ça ne serait pas plutôt en utilisant un type où la longueur en u8 ?

[Éloïs]

Je parle d'optimiser le temps d'exécution, pas la mémoire. ArrayString permet d'éviter les heap allocations.