sup39/sqlx
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Dont use EXPLAIN in nullability check for QuestDB

Browse source
This commit is contained in:
Jakub Labor 2024-12-16 12:21:57 -05:00 • committed by Austin Bonander
parent 7d148bd96b
commit 20bccf8877
1 changed files with 6 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
sqlx-postgres/src/connection/describe.rs
View file

@ -486,18 +486,12 @@ WHERE rngtypid = $1
)
})?;
// If the server is CockroachDB or Materialize, skip this step (#1248).
if !self
.inner
.stream
.parameter_statuses
.contains_key("crdb_version")
&& !self
.inner
.stream
.parameter_statuses
.contains_key("mz_version")
{
// If the server doesn't support EXPLAIN statements, skip this step (#1248).
let parameter_statuses = &self.inner.stream.parameter_statuses;
let is_cockroachdb = parameter_statuses.contains_key("crdb_version");
let is_materialize = parameter_statuses.contains_key("mz_version");
let is_questdb = parameter_statuses.contains_key("questdb_version");
if !is_cockroachdb && !is_materialize && !is_questdb {
// patch up our null inference with data from EXPLAIN
let nullable_patch = self
.nullables_from_explain(stmt_id, meta.parameters.len())