Skip to content

Commit ea6456f

Browse files
committed
Clarify stale remove_stale.. docs and logs
.. we were already doing the right thing, but our docs were stale.
1 parent 683cb81 commit ea6456f

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
@@ -413,7 +413,9 @@ macro_rules! define_run_body {
413413
log_trace!($logger, "Pruning and persisting network graph.");
414414
network_graph.remove_stale_channels_and_tracking_with_time(duration_since_epoch.as_secs());
415415
} else {
416-
log_warn!($logger, "Not pruning network graph, consider enabling `std` or doing so manually with remove_stale_channels_and_tracking_with_time.");
416+
log_warn!($logger,
417+
"Not pruning network graph, consider implementing the fetch_time argument or calling remove_stale_channels_and_tracking_with_time manually."
418+
);
417419
log_trace!($logger, "Persisting network graph.");
418420
}
419421

@@ -606,11 +608,6 @@ use futures_util::{dummy_waker, OptionalSelector, Selector, SelectorOutput};
606608
///
607609
/// See [`BackgroundProcessor::start`] for information on which actions this handles.
608610
///
609-
/// Requires the `futures` feature. Note that while this method is available without the `std`
610-
/// feature, doing so will skip calling [`NetworkGraph::remove_stale_channels_and_tracking`],
611-
/// you should call [`NetworkGraph::remove_stale_channels_and_tracking_with_time`] regularly
612-
/// manually instead.
613-
///
614611
/// The `mobile_interruptable_platform` flag should be set if we're currently running on a
615612
/// mobile device, where we may need to check for interruption of the application regularly. If you
616613
/// are unsure, you should set the flag, as the performance impact of it is minimal unless there

0 commit comments

Comments
 (0)