Skip to content

Commit 9167a29

Browse files
committed
Clarify stale remove_stale.. docs and logs
.. we were already doing the right thing, but our docs were stale.
1 parent 83e9e80 commit 9167a29

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

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -417,7 +417,9 @@ macro_rules! define_run_body {
417417
log_trace!($logger, "Pruning and persisting network graph.");
418418
network_graph.remove_stale_channels_and_tracking_with_time(duration_since_epoch.as_secs());
419419
} else {
420-
log_warn!($logger, "Not pruning network graph, consider enabling `std` or doing so manually with remove_stale_channels_and_tracking_with_time.");
420+
log_warn!($logger,
421+
"Not pruning network graph, consider implementing the fetch_time argument or calling remove_stale_channels_and_tracking_with_time manually."
422+
);
421423
log_trace!($logger, "Persisting network graph.");
422424
}
423425

@@ -600,11 +602,6 @@ use futures_util::{dummy_waker, OptionalSelector, Selector, SelectorOutput};
600602
///
601603
/// See [`BackgroundProcessor::start`] for information on which actions this handles.
602604
///
603-
/// Requires the `futures` feature. Note that while this method is available without the `std`
604-
/// feature, doing so will skip calling [`NetworkGraph::remove_stale_channels_and_tracking`],
605-
/// you should call [`NetworkGraph::remove_stale_channels_and_tracking_with_time`] regularly
606-
/// manually instead.
607-
///
608605
/// The `mobile_interruptable_platform` flag should be set if we're currently running on a
609606
/// mobile device, where we may need to check for interruption of the application regularly. If you
610607
/// are unsure, you should set the flag, as the performance impact of it is minimal unless there

0 commit comments

Comments
 (0)