Deuxfleurs/garage
41
56
Fork 73
Code Issues 98 Pull requests 14 Releases 30 Wiki Activity

table: remove redundant tracing in insert_many
All checks were successful
continuous-integration/drone/pr Build is passing
Details
continuous-integration/drone/push Build is passing
Details

Browse source
This commit is contained in:
Alex 2023-12-11 14:57:42 +01:00
parent f8df90b79b
commit e4f493b481
Signed by: lx
GPG key ID: 0E496D15096376BE
1 changed files with 2 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/table/table.rs
View file

@ -239,9 +239,7 @@ impl<F: TableSchema, R: TableReplication> Table<F, R> {
// Build futures to actually perform each of the corresponding RPC calls // Build futures to actually perform each of the corresponding RPC calls
let call_futures = call_list.into_iter().map(|(node, entries)| { let call_futures = call_list.into_iter().map(|(node, entries)| {
let this = self.clone(); let this = self.clone();
let tracer = opentelemetry::global::tracer("garage"); async move {
let span = tracer.start(format!("RPC to {:?}", node));
let fut = async move {
let rpc = TableRpc::<F>::Update(entries); let rpc = TableRpc::<F>::Update(entries);
let resp = this let resp = this
.system .system
@ -254,8 +252,7 @@ impl<F: TableSchema, R: TableReplication> Table<F, R> {
) )
.await; .await;
(node, resp) (node, resp)
}; }
fut.with_context(Context::current_with_span(span))
}); });
// Run all requests in parallel thanks to FuturesUnordered, and collect results. // Run all requests in parallel thanks to FuturesUnordered, and collect results.