Skip to content

Commit 406b04d

Browse files
committed
Introduce additional logging during serialization.
1 parent 63edf99 commit 406b04d

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ async fn serialize_delta<L: Deref + Clone>(network_graph: Arc<NetworkGraph<L>>,
193193
log_info!(logger, "update-fetched channel count: {}", delta_set.len());
194194
lookup::filter_delta_set(&mut delta_set, logger.clone());
195195
log_info!(logger, "update-filtered channel count: {}", delta_set.len());
196-
let serialization_details = serialization::serialize_delta_set(delta_set, last_sync_timestamp);
196+
let serialization_details = serialization::serialize_delta_set(delta_set, last_sync_timestamp, logger.clone());
197197

198198
// process announcements
199199
// write the number of channel announcements to the output

src/serialization.rs

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,11 @@
11
use std::cmp::max;
22
use std::collections::HashMap;
3+
use std::ops::Deref;
34

45
use bitcoin::BlockHash;
56
use lightning::ln::msgs::{UnsignedChannelAnnouncement, UnsignedChannelUpdate};
7+
use lightning::log_trace;
8+
use lightning::util::logger::Logger;
69
use lightning::util::ser::{BigSize, Writeable};
710

811
use crate::config;
@@ -102,7 +105,7 @@ struct FullUpdateValueHistograms {
102105
htlc_maximum_msat: HashMap<u64, usize>,
103106
}
104107

105-
pub(super) fn serialize_delta_set(delta_set: DeltaSet, last_sync_timestamp: u32) -> SerializationSet {
108+
pub(super) fn serialize_delta_set<L: Deref>(delta_set: DeltaSet, last_sync_timestamp: u32, logger: L) -> SerializationSet where L::Target: Logger {
106109
let chain_hash = bitcoin::blockdata::constants::genesis_block(config::network()).block_hash();
107110
let mut serialization_set = SerializationSet {
108111
announcements: vec![],
@@ -135,10 +138,14 @@ pub(super) fn serialize_delta_set(delta_set: DeltaSet, last_sync_timestamp: u32)
135138
let current_announcement_seen = channel_announcement_delta.seen;
136139
let is_new_announcement = current_announcement_seen >= last_sync_timestamp;
137140
let is_newly_included_announcement = if let Some(first_update_seen) = channel_delta.first_bidirectional_updates_seen {
141+
#[cfg(test)]
142+
log_trace!(logger, "Channel {} first bidirectional update seen: {}", scid, first_update_seen);
138143
first_update_seen >= last_sync_timestamp
139144
} else {
140145
false
141146
};
147+
#[cfg(test)]
148+
log_trace!(logger, "Channel {} announcement seen at {} (new: {}, newly included: {})", scid, current_announcement_seen, is_new_announcement, is_newly_included_announcement);
142149
let send_announcement = is_new_announcement || is_newly_included_announcement;
143150
if send_announcement {
144151
serialization_set.latest_seen = max(serialization_set.latest_seen, current_announcement_seen);

0 commit comments

Comments
 (0)