Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Commit

Permalink
Remove warning about networking taking a lot of time (#6630)
Browse files Browse the repository at this point in the history
* Remove warning about networking taking a lot of time

* Fix warnings
  • Loading branch information
tomaka authored Jul 10, 2020
1 parent 524c76f commit 3b1e35d
Showing 1 changed file with 2 additions and 14 deletions.
16 changes: 2 additions & 14 deletions client/service/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -38,13 +38,12 @@ use std::{io, pin::Pin};
use std::net::SocketAddr;
use std::collections::HashMap;
use std::time::Duration;
use wasm_timer::Instant;
use std::task::Poll;
use parking_lot::Mutex;

use futures::{Future, FutureExt, Stream, StreamExt, stream, compat::*};
use sc_network::{NetworkStatus, network_state::NetworkState, PeerId};
use log::{log, warn, debug, error, Level};
use log::{warn, debug, error};
use codec::{Encode, Decode};
use sp_runtime::generic::BlockId;
use sp_runtime::traits::{Block as BlockT, Header as HeaderT};
Expand Down Expand Up @@ -156,7 +155,7 @@ pub struct ServiceComponents<TBl: BlockT, TBackend: Backend<TBl>, TSc, TExPool,
pub client: Arc<TCl>,
/// A shared transaction pool instance.
pub transaction_pool: Arc<TExPool>,
/// The chain task manager.
/// The chain task manager.
pub task_manager: TaskManager,
/// A keystore that stores keys.
pub keystore: sc_keystore::KeyStorePtr,
Expand Down Expand Up @@ -216,8 +215,6 @@ async fn build_network_future<
};

loop {
let before_polling = Instant::now();

futures::select!{
// List of blocks that the client has imported.
notification = imported_blocks_stream.next() => {
Expand Down Expand Up @@ -334,15 +331,6 @@ async fn build_network_future<
ready_sink.send((status, state));
}
}

// Now some diagnostic for performances.
let polling_dur = before_polling.elapsed();
log!(
target: "service",
if polling_dur >= Duration::from_secs(1) { Level::Warn } else { Level::Trace },
"⚠️ Polling the network future took {:?}",
polling_dur
);
}
}

Expand Down

0 comments on commit 3b1e35d

Please sign in to comment.