Skip to content

Commit 8091162

Browse files
committed
Introduce additional logging during serialization.
1 parent 91ab27c commit 8091162

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
@@ -198,7 +198,7 @@ async fn serialize_delta<L: Deref + Clone>(network_graph: Arc<NetworkGraph<L>>,
198198
log_info!(logger, "update-fetched channel count: {}", delta_set.len());
199199
lookup::filter_delta_set(&mut delta_set, logger.clone());
200200
log_info!(logger, "update-filtered channel count: {}", delta_set.len());
201-
let serialization_details = serialization::serialize_delta_set(delta_set, last_sync_timestamp);
201+
let serialization_details = serialization::serialize_delta_set(delta_set, last_sync_timestamp, logger.clone());
202202

203203
// process announcements
204204
// 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,9 +1,12 @@
11
use std::cmp::max;
22
use std::collections::HashMap;
33
use std::time::{SystemTime, UNIX_EPOCH};
4+
use std::ops::Deref;
45

56
use bitcoin::BlockHash;
67
use lightning::ln::msgs::{UnsignedChannelAnnouncement, UnsignedChannelUpdate};
8+
use lightning::log_trace;
9+
use lightning::util::logger::Logger;
710
use lightning::util::ser::{BigSize, Writeable};
811
use crate::config;
912

@@ -104,7 +107,7 @@ struct FullUpdateValueHistograms {
104107
htlc_maximum_msat: HashMap<u64, usize>,
105108
}
106109

107-
pub(super) fn serialize_delta_set(delta_set: DeltaSet, last_sync_timestamp: u32) -> SerializationSet {
110+
pub(super) fn serialize_delta_set<L: Deref>(delta_set: DeltaSet, last_sync_timestamp: u32, logger: L) -> SerializationSet where L::Target: Logger {
108111
let chain_hash = bitcoin::blockdata::constants::genesis_block(config::network()).block_hash();
109112
let mut serialization_set = SerializationSet {
110113
announcements: vec![],
@@ -140,10 +143,14 @@ pub(super) fn serialize_delta_set(delta_set: DeltaSet, last_sync_timestamp: u32)
140143
let current_announcement_seen = channel_announcement_delta.seen;
141144
let is_new_announcement = current_announcement_seen >= last_sync_timestamp;
142145
let is_newly_included_announcement = if let Some(first_update_seen) = channel_delta.first_bidirectional_updates_seen {
146+
#[cfg(test)]
147+
log_trace!(logger, "Channel {} first bidirectional update seen: {}", scid, first_update_seen);
143148
first_update_seen >= last_sync_timestamp
144149
} else {
145150
false
146151
};
152+
#[cfg(test)]
153+
log_trace!(logger, "Channel {} announcement seen at {} (new: {}, newly included: {})", scid, current_announcement_seen, is_new_announcement, is_newly_included_announcement);
147154
let send_announcement = is_new_announcement || is_newly_included_announcement;
148155
if send_announcement {
149156
serialization_set.latest_seen = max(serialization_set.latest_seen, current_announcement_seen);

0 commit comments

Comments
 (0)