Skip to content

Commit c0c948f

Browse files
authored
Bug, warn on batch rejected by the da sequencer. (#1249)
1 parent 6619eeb commit c0c948f

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

protocol-units/da-sequencer/node/src/server.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,10 @@ pub enum GrpcRequests {
7070
pub struct DaSequencerNode {
7171
request_tx: mpsc::Sender<GrpcRequests>,
7272
whitelist: Whitelist,
73+
// State propagation is not a main functionality of the da-sequencer. It's used to detect instabilities.
74+
// `main_node_verifying_key` contains the main node signing public key for state propagation.
75+
// It's optional because issue on the mainnode can stop the network.
76+
// To be able to recover fast, disabling it can be useful.
7377
main_node_verifying_key: Option<VerifyingKey>,
7478
}
7579

protocol-units/execution/maptos/opt-executor/src/background/transaction_pipe.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,8 +177,7 @@ impl TransactionPipe {
177177
Ok(Ok(response)) => {
178178
debug!("After sent batch.");
179179
if !response.answer {
180-
tracing::error!("DA Sequencer reject batch, can't send batch, exit process");
181-
return Err(Error::InternalError(format!("DA Sequencer reject batch, can't send batch, exit process")));
180+
tracing::warn!("DA Sequencer reject batch, can't send the batch.");
182181
}
183182
}
184183
Ok(Err(err)) => {

0 commit comments

Comments
 (0)