address comments

aptos-labs · Jan 12, 2025 · 67da8b9 · 67da8b9
1 parent 2f78e26
commit 67da8b9
Show file tree

Hide file tree

Showing 8 changed files with 523 additions and 2,147 deletions.
diff --git a/aptos-move/e2e-benchmark/data/calibration_values.tsv b/aptos-move/e2e-benchmark/data/calibration_values.tsv
@@ -33,11 +33,6 @@ VectorRangeMove { vec_len: 3000, element_len: 1, index: 1000, move_len: 500, rep
 VectorTrimAppend { vec_len: 100, element_len: 100, index: 0, repeats: 0 }	119	0.909	1.201	293.5
 VectorTrimAppend { vec_len: 100, element_len: 100, index: 10, repeats: 1000 }	119	0.951	1.143	12571.2
 VectorRangeMove { vec_len: 100, element_len: 100, index: 50, move_len: 10, repeats: 1000 }	6	0.925	1.001	5316.2
-MapInsertRemove { len: 10, repeats: 0, map_type: OrderedMap }	6	0.925	1.001	378
-MapInsertRemove { len: 10, repeats: 100, map_type: OrderedMap }	6	0.925	1.001	8184
-MapInsertRemove { len: 10, repeats: 100, map_type: SimpleMap }	6	0.925	1.001	8201.7
-MapInsertRemove { len: 100, repeats: 0, map_type: OrderedMap }	6	0.925	1.001	5094
 MapInsertRemove { len: 100, repeats: 100, map_type: OrderedMap }	6	0.925	1.001	15838
 MapInsertRemove { len: 100, repeats: 100, map_type: SimpleMap }	6	0.925	1.001	39802.4
-MapInsertRemove { len: 1000, repeats: 0, map_type: OrderedMap }	6	0.925	1.001	66878
 MapInsertRemove { len: 1000, repeats: 100, map_type: OrderedMap }	6	0.925	1.001	79826
diff --git a/aptos-move/e2e-benchmark/src/main.rs b/aptos-move/e2e-benchmark/src/main.rs
@@ -8,13 +8,8 @@ use aptos_language_e2e_tests::{
 use aptos_transaction_generator_lib::{
     entry_point_trait::{AutomaticArgs, EntryPointTrait, MultiSigConfig},
     publishing::publish_util::{Package, PackageHandler},
-    publishing::{
-        module_simple::{AutomaticArgs, LoopType, MapType, MultiSigConfig},
-        publish_util::{Package, PackageHandler},
-    },
-    EntryPoints,
 };
-use aptos_transaction_workloads_lib::{EntryPoints, LoopType};
+use aptos_transaction_workloads_lib::{EntryPoints, LoopType, MapType};
 use aptos_types::{account_address::AccountAddress, transaction::TransactionPayload};
 use rand::{rngs::StdRng, SeedableRng};
 use serde_json::json;
@@ -243,26 +238,6 @@ fn main() {
             move_len: 10,
             repeats: 1000,
         },
-        EntryPoints::MapInsertRemove {
-            len: 10,
-            repeats: 0,
-            map_type: MapType::OrderedMap,
-        },
-        EntryPoints::MapInsertRemove {
-            len: 10,
-            repeats: 100,
-            map_type: MapType::OrderedMap,
-        },
-        EntryPoints::MapInsertRemove {
-            len: 10,
-            repeats: 100,
-            map_type: MapType::SimpleMap,
-        },
-        EntryPoints::MapInsertRemove {
-            len: 100,
-            repeats: 0,
-            map_type: MapType::OrderedMap,
-        },
         EntryPoints::MapInsertRemove {
             len: 100,
             repeats: 100,
@@ -273,11 +248,6 @@ fn main() {
             repeats: 100,
             map_type: MapType::SimpleMap,
         },
-        EntryPoints::MapInsertRemove {
-            len: 1000,
-            repeats: 0,
-            map_type: MapType::OrderedMap,
-        },
         EntryPoints::MapInsertRemove {
             len: 1000,
             repeats: 100,

diff --git a/aptos-move/framework/aptos-stdlib/doc/big_ordered_map.md b/aptos-move/framework/aptos-stdlib/doc/big_ordered_map.md