Skip to content

Commit 983b32c

Browse files
committed
sim-rs: remove tx-sharded-fraction setting
1 parent 5b4b7c6 commit 983b32c

File tree

8 files changed

+6
-24
lines changed

8 files changed

+6
-24
lines changed

data/simulation/config.d.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -63,11 +63,6 @@ export interface Config {
6363
"tx-generation-distribution": Distribution;
6464
/** Only supported by Rust simulation. */
6565
"tx-size-bytes-distribution": Distribution;
66-
/**
67-
* What fraction of transactions have at least one sharded input?
68-
*
69-
* Only supported by Rust simulation. */
70-
"tx-sharded-fraction": number;
7166
/** Only supported by Rust simulation. */
7267
"tx-validation-cpu-time-ms": number;
7368
/** Only supported by Rust simulation. */

data/simulation/config.default.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ tx-size-bytes-distribution:
4444
distribution: log-normal
4545
mu: 6.833
4646
sigma: 1.127
47-
tx-sharded-fraction: 0.5
4847
tx-validation-cpu-time-ms: 1.5
4948
tx-max-size-bytes: 16384
5049

data/simulation/config.schema.json

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -353,10 +353,6 @@
353353
"properties": {},
354354
"type": "number"
355355
},
356-
"tx-sharded-fraction": {
357-
"description": "What fraction of transactions have at least one sharded input?\n\nOnly supported by Rust simulation.",
358-
"type": "number"
359-
},
360356
"tx-size-bytes-distribution": {
361357
"$ref": "#/definitions/Distribution",
362358
"description": "Only supported by Rust simulation."

sim-rs/sim-core/src/config.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,6 @@ pub struct RawParameters {
6868
// Transaction configuration
6969
pub tx_generation_distribution: DistributionConfig,
7070
pub tx_size_bytes_distribution: DistributionConfig,
71-
pub tx_sharded_fraction: f64,
7271
pub tx_validation_cpu_time_ms: f64,
7372
pub tx_max_size_bytes: u64,
7473
pub tx_start_time: Option<f64>,
@@ -379,7 +378,6 @@ impl TransactionConfig {
379378
if params.simulate_transactions {
380379
Self::Real(RealTransactionConfig {
381380
max_size: params.tx_max_size_bytes,
382-
sharded_fraction: params.tx_sharded_fraction,
383381
frequency_ms: params.tx_generation_distribution.into(),
384382
size_bytes: params.tx_size_bytes_distribution.into(),
385383
start_time: params
@@ -402,7 +400,6 @@ impl TransactionConfig {
402400
#[derive(Debug, Clone)]
403401
pub(crate) struct RealTransactionConfig {
404402
pub max_size: u64,
405-
pub sharded_fraction: f64,
406403
pub frequency_ms: FloatDistribution,
407404
pub size_bytes: FloatDistribution,
408405
pub start_time: Option<Timestamp>,

sim-rs/sim-core/src/events.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ pub enum Event {
102102
id: TransactionId,
103103
publisher: Node,
104104
size_bytes: u64,
105-
shard: Option<u64>,
105+
shard: u64,
106106
},
107107
TXSent {
108108
id: TransactionId,

sim-rs/sim-core/src/model.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ id_wrapper!(TransactionId, u64);
8989
#[derive(Debug, Clone, PartialEq, Eq)]
9090
pub struct Transaction {
9191
pub id: TransactionId,
92-
pub shard: Option<u64>,
92+
pub shard: u64,
9393
pub bytes: u64,
9494
}
9595

sim-rs/sim-core/src/sim/node.rs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -748,7 +748,7 @@ impl Node {
748748
// Add one transaction, the right size for the extra RB payload
749749
let tx = Transaction {
750750
id: config.next_id(),
751-
shard: None,
751+
shard: 0,
752752
bytes: config.rb_size,
753753
};
754754
self.tracker.track_transaction_generated(&tx, self.id);
@@ -1357,16 +1357,13 @@ impl Node {
13571357
if let TransactionConfig::Mock(config) = &self.sim_config.transactions {
13581358
let tx = Transaction {
13591359
id: config.next_id(),
1360-
shard: None,
1360+
shard,
13611361
bytes: config.ib_size,
13621362
};
13631363
self.tracker.track_transaction_generated(&tx, self.id);
13641364
vec![Arc::new(tx)]
13651365
} else {
1366-
self.select_txs(
1367-
|seen| seen.tx.shard.is_none_or(|s| s == shard),
1368-
self.sim_config.max_ib_size,
1369-
)
1366+
self.select_txs(|seen| seen.tx.shard == shard, self.sim_config.max_ib_size)
13701367
}
13711368
}
13721369

sim-rs/sim-core/src/sim/tx.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,7 @@ impl TransactionProducer {
5555

5656
loop {
5757
let id = TransactionId::new(next_tx_id);
58-
let shard = rng
59-
.random_bool(config.sharded_fraction)
60-
.then(|| rng.random_range(0..self.ib_shards));
58+
let shard = rng.random_range(0..self.ib_shards);
6159
let bytes = (config.size_bytes.sample(&mut rng) as u64).min(config.max_size);
6260
let tx = Transaction { id, shard, bytes };
6361

0 commit comments

Comments
 (0)