Skip to content

Commit cf1da4d

Browse files
committed
pgwire: keep query-terminated wording for dropped dependency in FETCH
The FETCH path rendered `PeekResponseUnary::DependencyDropped` via `to_concurrent_dependency_drop()`, changing the message from "query could not complete because relation ... was dropped" to "relation '...' was dropped". Restore `query_terminated_error()` so the message and SQLSTATE are unchanged; this rendering is out of scope for the evaluation-error SQLSTATE work and broke `test_subscribe_on_dropped_source`.
1 parent 37da14b commit cf1da4d

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

‎src/pgwire/src/protocol.rs‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2414,9 +2414,10 @@ where
24142414
FetchResult::Error(err.into_response(Severity::Error))
24152415
}
24162416
Some(PeekResponseUnary::DependencyDropped(dep)) => {
2417-
FetchResult::Error(
2418-
dep.to_concurrent_dependency_drop().into_response(Severity::Error),
2419-
)
2417+
FetchResult::Error(ErrorResponse::error(
2418+
SqlState::INTERNAL_ERROR,
2419+
dep.query_terminated_error(),
2420+
))
24202421
}
24212422
Some(PeekResponseUnary::Canceled) => FetchResult::Canceled,
24222423
},

0 commit comments

Comments
 (0)