mirror of
https://github.com/0glabs/0g-storage-node.git
synced 2025-11-03 08:07:27 +00:00
fill all middle seqs
This commit is contained in:
parent
1381428aee
commit
b05753b06f
@ -319,11 +319,11 @@ impl LogStoreWrite for LogManager {
|
||||
// If this is the first tx with this data root, copy and finalize all same-root txs.
|
||||
self.copy_tx_and_finalize(tx_seq, same_root_seq_list[1..].to_vec())?;
|
||||
} else {
|
||||
// If this is not the first tx with this data root, copy and finalize the first one.
|
||||
// If this is not the first tx with this data root, and the first one is not finalized.
|
||||
let maybe_first_seq = same_root_seq_list.first().cloned();
|
||||
if let Some(first_seq) = maybe_first_seq {
|
||||
if !self.check_tx_completed(first_seq)? {
|
||||
self.copy_tx_and_finalize(tx_seq, vec![first_seq])?;
|
||||
self.copy_tx_and_finalize(tx_seq, same_root_seq_list)?;
|
||||
}
|
||||
}
|
||||
}
|
||||
@ -357,7 +357,7 @@ impl LogStoreWrite for LogManager {
|
||||
// TODO: Should we double check the tx merkle root?
|
||||
let tx_end_index = tx.start_entry_index + bytes_to_entries(tx.size);
|
||||
if self.check_data_completed(tx.start_entry_index, tx_end_index)? {
|
||||
self.tx_store.finalize_tx(tx_seq)?;
|
||||
|
||||
let same_root_seq_list = self
|
||||
.tx_store
|
||||
.get_tx_seq_list_by_data_root(&tx.data_merkle_root)?;
|
||||
@ -370,11 +370,13 @@ impl LogStoreWrite for LogManager {
|
||||
let maybe_first_seq = same_root_seq_list.first().cloned();
|
||||
if let Some(first_seq) = maybe_first_seq {
|
||||
if !self.check_tx_completed(first_seq)? {
|
||||
self.copy_tx_and_finalize(tx_seq, vec![first_seq])?;
|
||||
self.copy_tx_and_finalize(tx_seq, same_root_seq_list)?;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
self.tx_store.finalize_tx(tx_seq)?;
|
||||
|
||||
metrics::FINALIZE_TX_WITH_HASH.update_since(start_time);
|
||||
Ok(true)
|
||||
} else {
|
||||
@ -1194,8 +1196,8 @@ impl LogManager {
|
||||
let mut to_tx_offset_list = Vec::with_capacity(to_tx_seq_list.len());
|
||||
|
||||
for seq in to_tx_seq_list {
|
||||
// No need to copy data for completed tx.
|
||||
if self.check_tx_completed(seq)? {
|
||||
// No need to copy data for completed tx and itself
|
||||
if self.check_tx_completed(seq)? || from_tx_seq == seq {
|
||||
continue;
|
||||
}
|
||||
let tx = self
|
||||
|
||||
Loading…
Reference in New Issue
Block a user