Skip to content

Commit 868eb44

Browse files
committed
fix
1 parent 09b0ed9 commit 868eb44

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

src/query/service/src/servers/http/v1/http_query_handlers.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -310,6 +310,7 @@ async fn query_final_handler(
310310
&ctx.client_session_id,
311311
StopReason::Finished,
312312
ErrorCode::ClosedQuery("Query closed by client"),
313+
true
313314
)
314315
.await?
315316
{
@@ -351,6 +352,7 @@ async fn query_cancel_handler(
351352
&ctx.client_session_id,
352353
StopReason::Canceled,
353354
ErrorCode::AbortedQuery("canceled by client"),
355+
true
354356
)
355357
.await?
356358
{

src/query/service/src/servers/http/v1/query/http_query_manager.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,6 +189,7 @@ impl HttpQueryManager {
189189
&None,
190190
stop_reason,
191191
ErrorCode::AbortedQuery(&msg),
192+
false
192193
)
193194
.await
194195
.ok();
@@ -214,14 +215,15 @@ impl HttpQueryManager {
214215
client_session_id: &Option<String>,
215216
reason: StopReason,
216217
error: ErrorCode,
218+
check_session_id: bool,
217219
) -> poem::error::Result<Option<Arc<HttpQuery>>> {
218220
let now = SystemTime::now()
219221
.duration_since(SystemTime::UNIX_EPOCH)
220222
.expect("Time went backwards")
221223
.as_secs();
222224
let (query, stop_first_run) = self.queries.write().stop(query_id, reason, now);
223225
if let Some(q) = &query {
224-
if reason != StopReason::Timeout {
226+
if check_session_id {
225227
q.check_client_session_id(client_session_id)?;
226228
}
227229
if stop_first_run {

0 commit comments

Comments
 (0)