Merge branch 'main' into dom/wal-syscalls

pull/24376/head
Dom 2023-02-23 20:21:53 +00:00 committed by GitHub
commit f19fbc2d88
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 15 additions and 3 deletions

View File

@ -166,7 +166,7 @@ impl<T> QueryResultInstrumentation<T> {
.register_metric_with_options::<U64Histogram, _>(
"ingester_query_result_partition",
"distribution of query result partition count sent to the client",
|| U64HistogramOptions::new([1, 2, 3, 4, 5]),
|| U64HistogramOptions::new([1, 2, 3, 4, 5, 6, 7, 8, 9, 10]),
)
.recorder(&[]);

View File

@ -176,7 +176,7 @@ where
warn!(predicate=?p, "ignoring query predicate (unsupported)");
}
let response = self
let response = match self
.query_handler
.query_exec(
namespace_id,
@ -184,7 +184,19 @@ where
request.columns,
span_ctx.child_span("ingester query"),
)
.await?;
.await
{
Ok(v) => v,
Err(e) => {
error!(
error=%e,
%namespace_id,
%table_id,
"query error"
);
return Err(e)?;
}
};
let output = encode_response(response, self.ingester_id).map_err(tonic::Status::from);