itsscb/sqlx
1
0
Fork 0
mirror of https://github.com/launchbadge/sqlx.git synced 2025-10-02 15:25:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

Don't run EXPLAIN nullability analysis on Materialize

Browse Source 
Materialize [0] is a PostgreSQL-like database that, similar to
CockroachDB, does not support PostgreSQL's `EXPLAIN` output. Extend the
fix from PR #1248 to Materialize, too, so that sqlx can still be used
with Materialize.

See #1248.

[0]: https://materialize.com
This commit is contained in:
Nikhil Benesch 2023-01-07 20:47:10 -05:00 committed by Austin Bonander
parent c4b835c23a
commit bb2baf2183
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
sqlx-postgres/src/connection/describe.rs
View File

@ -399,8 +399,10 @@ WHERE rngtypid = $1
.fetch_all(&mut *self) .fetch_all(&mut *self)
.await?; .await?;
// if it's cockroachdb skip this step #1248 // If the server is CockroachDB or Materialize, skip this step (#1248).
if !self.stream.parameter_statuses.contains_key("crdb_version") { if !self.stream.parameter_statuses.contains_key("crdb_version")
&& !self.stream.parameter_statuses.contains_key("mz_version")
{
// patch up our null inference with data from EXPLAIN // patch up our null inference with data from EXPLAIN
let nullable_patch = self let nullable_patch = self
.nullables_from_explain(stmt_id, meta.parameters.len()) .nullables_from_explain(stmt_id, meta.parameters.len())