Skip to content

Commit 4af5fc3

Browse files
committed
Clarify stale remove_stale.. docs and logs
.. we were already doing the right thing, but our docs were stale.
1 parent 0820a0e commit 4af5fc3

File tree

1 file changed

+3
-6
lines changed
  • lightning-background-processor/src

1 file changed

+3
-6
lines changed

lightning-background-processor/src/lib.rs

+3-6
Original file line numberDiff line numberDiff line change
@@ -419,7 +419,9 @@ macro_rules! define_run_body {
419419
log_trace!($logger, "Pruning and persisting network graph.");
420420
network_graph.remove_stale_channels_and_tracking_with_time(duration_since_epoch.as_secs());
421421
} else {
422-
log_warn!($logger, "Not pruning network graph, consider enabling `std` or doing so manually with remove_stale_channels_and_tracking_with_time.");
422+
log_warn!($logger,
423+
"Not pruning network graph, consider implementing the fetch_time argument or calling remove_stale_channels_and_tracking_with_time manually."
424+
);
423425
log_trace!($logger, "Persisting network graph.");
424426
}
425427

@@ -612,11 +614,6 @@ use futures_util::{dummy_waker, OptionalSelector, Selector, SelectorOutput};
612614
///
613615
/// See [`BackgroundProcessor::start`] for information on which actions this handles.
614616
///
615-
/// Requires the `futures` feature. Note that while this method is available without the `std`
616-
/// feature, doing so will skip calling [`NetworkGraph::remove_stale_channels_and_tracking`],
617-
/// you should call [`NetworkGraph::remove_stale_channels_and_tracking_with_time`] regularly
618-
/// manually instead.
619-
///
620617
/// The `mobile_interruptable_platform` flag should be set if we're currently running on a
621618
/// mobile device, where we may need to check for interruption of the application regularly. If you
622619
/// are unsure, you should set the flag, as the performance impact of it is minimal unless there

0 commit comments

Comments
 (0)