Add query_opt

Closes rust-postgres#510

Author: sfackler

postgres-native-tls/src/lib.rs

@@ -48,17 +48,17 @@
 #![doc(html_root_url = "https://docs.rs/postgres-native-tls/0.3")]
 #![warn(rust_2018_idioms, clippy::all, missing_docs)]
 
-use std::task::{Context, Poll};
+use bytes::{Buf, BufMut};
 use std::future::Future;
 use std::io;
+use std::mem::MaybeUninit;
 use std::pin::Pin;
+use std::task::{Context, Poll};
 use tokio::io::{AsyncRead, AsyncWrite};
-use bytes::{Buf, BufMut};
 use tokio_postgres::tls;
 #[cfg(feature = "runtime")]
 use tokio_postgres::tls::MakeTlsConnect;
 use tokio_postgres::tls::{ChannelBinding, TlsConnect};
-use std::mem::MaybeUninit;
 
 #[cfg(test)]
 mod test;

postgres-openssl/src/lib.rs

@@ -42,7 +42,7 @@
 #![doc(html_root_url = "https://docs.rs/postgres-openssl/0.3")]
 #![warn(rust_2018_idioms, clippy::all, missing_docs)]
 
-use std::task::{Poll, Context};
+use bytes::{Buf, BufMut};
 #[cfg(feature = "runtime")]
 use openssl::error::ErrorStack;
 use openssl::hash::MessageDigest;
@@ -53,17 +53,17 @@ use openssl::ssl::{ConnectConfiguration, SslRef};
 use std::fmt::Debug;
 use std::future::Future;
 use std::io;
+use std::mem::MaybeUninit;
 use std::pin::Pin;
 #[cfg(feature = "runtime")]
 use std::sync::Arc;
+use std::task::{Context, Poll};
 use tokio::io::{AsyncRead, AsyncWrite};
-use bytes::{Buf, BufMut};
 use tokio_openssl::{HandshakeError, SslStream};
 use tokio_postgres::tls;
 #[cfg(feature = "runtime")]
 use tokio_postgres::tls::MakeTlsConnect;
 use tokio_postgres::tls::{ChannelBinding, TlsConnect};
-use std::mem::MaybeUninit;
 
 #[cfg(test)]
 mod test;

postgres/src/client.rs

@@ -119,6 +119,8 @@ impl Client {
 
     /// Executes a statement which returns a single row, returning it.
     ///
+    /// Returns an error if the query does not return exactly one row.
+    ///
     /// A statement may contain parameters, specified by `$n`, where `n` is the index of the parameter of the list
     /// provided, 1-indexed.
     ///
@@ -152,6 +154,52 @@ impl Client {
         executor::block_on(self.0.query_one(query, params))
     }
 
+    /// Executes a statement which returns zero or one rows, returning it.
+    ///
+    /// Returns an error if the query returns more than one row.
+    ///
+    /// A statement may contain parameters, specified by `$n`, where `n` is the index of the parameter of the list
+    /// provided, 1-indexed.
+    ///
+    /// The `query` argument can either be a `Statement`, or a raw query string. If the same statement will be
+    /// repeatedly executed (perhaps with different query parameters), consider preparing the statement up front
+    /// with the `prepare` method.
+    ///
+    /// # Panics
+    ///
+    /// Panics if the number of parameters provided does not match the number expected.
+    ///
+    /// # Examples
+    ///
+    /// ```no_run
+    /// use postgres::{Client, NoTls};
+    ///
+    /// # fn main() -> Result<(), postgres::Error> {
+    /// let mut client = Client::connect("host=localhost user=postgres", NoTls)?;
+    ///
+    /// let baz = true;
+    /// let row = client.query_opt("SELECT foo FROM bar WHERE baz = $1", &[&baz])?;
+    /// match row {
+    ///     Some(row) => {
+    ///         let foo: i32 = row.get("foo");
+    ///         println!("foo: {}", foo);
+    ///     }
+    ///     None => println!("no matching foo"),
+    /// }
+    /// # Ok(())
+    /// # }
+    /// ```
+    pub fn query_opt<T>(
+        &mut self,
+        query: &T,
+        params: &[&(dyn ToSql + Sync)],
+    ) -> Result<Option<Row>, Error>
+    where
+        T: ?Sized + ToStatement,
+    {
+        executor::block_on(self.0.query_opt(query, params))
+    }
+
     /// A maximally-flexible version of `query`.
     ///
     /// It takes an iterator of parameters rather than a slice, and returns an iterator of rows rather than collecting

postgres/src/transaction.rs

@@ -60,6 +60,18 @@ impl<'a> Transaction<'a> {
         executor::block_on(self.0.query_one(query, params))
     }
 
+    /// Like `Client::query_opt`.
+    pub fn query_opt<T>(
+        &mut self,
+        query: &T,
+        params: &[&(dyn ToSql + Sync)],
+    ) -> Result<Option<Row>, Error>
+    where
+        T: ?Sized + ToStatement,
+    {
+        executor::block_on(self.0.query_opt(query, params))
+    }
+
     /// Like `Client::query_raw`.
     pub fn query_raw<'b, T, I>(&mut self, query: &T, params: I) -> Result<RowIter<'_>, Error>
     where

tokio-postgres-binary-copy/src/lib.rs

@@ -1,16 +1,16 @@
-use bytes::{BufMut, Bytes, BytesMut, Buf};
-use futures::{ready, Stream, SinkExt};
+use byteorder::{BigEndian, ByteOrder};
+use bytes::{Buf, BufMut, Bytes, BytesMut};
+use futures::{ready, SinkExt, Stream};
 use pin_project_lite::pin_project;
 use std::convert::TryFrom;
 use std::error::Error;
+use std::io::Cursor;
 use std::ops::Range;
 use std::pin::Pin;
 use std::sync::Arc;
 use std::task::{Context, Poll};
-use tokio_postgres::types::{IsNull, ToSql, Type, FromSql, WrongType};
-use tokio_postgres::{CopyOutStream, CopyInSink};
-use std::io::Cursor;
-use byteorder::{ByteOrder, BigEndian};
+use tokio_postgres::types::{FromSql, IsNull, ToSql, Type, WrongType};
+use tokio_postgres::{CopyInSink, CopyOutStream};
 
 #[cfg(test)]
 mod test;
@@ -49,10 +49,13 @@ impl BinaryCopyInWriter {
         self.write_raw(values.iter().cloned()).await
     }
 
-    pub async fn write_raw<'a, I>(self: Pin<&mut Self>, values: I) -> Result<(), Box<dyn Error + Sync + Send>>
-        where
-            I: IntoIterator<Item = &'a (dyn ToSql + Send)>,
-            I::IntoIter: ExactSizeIterator,
+    pub async fn write_raw<'a, I>(
+        self: Pin<&mut Self>,
+        values: I,
+    ) -> Result<(), Box<dyn Error + Sync + Send>>
+    where
+        I: IntoIterator<Item = &'a (dyn ToSql + Send)>,
+        I::IntoIter: ExactSizeIterator,
     {
         let mut this = self.project();
 
@@ -126,7 +129,7 @@ impl Stream for BinaryCopyOutStream {
             Some(Err(e)) => return Poll::Ready(Some(Err(e.into()))),
             None => return Poll::Ready(Some(Err("unexpected EOF".into()))),
         };
-        let mut chunk= Cursor::new(chunk);
+        let mut chunk = Cursor::new(chunk);
 
         let has_oids = match &this.header {
             Some(header) => header.has_oids,
@@ -200,19 +203,25 @@ pub struct BinaryCopyOutRow {
 }
 
 impl BinaryCopyOutRow {
-    pub fn try_get<'a, T>(&'a self, idx: usize) -> Result<T, Box<dyn Error + Sync + Send>> where T: FromSql<'a> {
+    pub fn try_get<'a, T>(&'a self, idx: usize) -> Result<T, Box<dyn Error + Sync + Send>>
+    where
+        T: FromSql<'a>,
+    {
         let type_ = &self.types[idx];
         if !T::accepts(type_) {
             return Err(WrongType::new::<T>(type_.clone()).into());
         }
 
         match &self.ranges[idx] {
             Some(range) => T::from_sql(type_, &self.buf[range.clone()]).map_err(Into::into),
-            None => T::from_sql_null(type_).map_err(Into::into)
+            None => T::from_sql_null(type_).map_err(Into::into),
         }
     }
 
-    pub fn get<'a, T>(&'a self, idx: usize) -> T where T: FromSql<'a> {
+    pub fn get<'a, T>(&'a self, idx: usize) -> T
+    where
+        T: FromSql<'a>,
+    {
         match self.try_get(idx) {
             Ok(value) => value,
             Err(e) => panic!("error retrieving column {}: {}", idx, e),

tokio-postgres-binary-copy/src/test.rs

@@ -1,7 +1,7 @@
 use crate::{BinaryCopyInWriter, BinaryCopyOutStream};
+use futures::{pin_mut, TryStreamExt};
 use tokio_postgres::types::Type;
 use tokio_postgres::{Client, NoTls};
-use futures::{TryStreamExt, pin_mut};
 
 async fn connect() -> Client {
     let (client, connection) =
@@ -23,11 +23,18 @@ async fn write_basic() {
         .await
         .unwrap();
 
-    let sink = client.copy_in("COPY foo (id, bar) FROM STDIN BINARY", &[]).await.unwrap();
+    let sink = client
+        .copy_in("COPY foo (id, bar) FROM STDIN BINARY", &[])
+        .await
+        .unwrap();
     let writer = BinaryCopyInWriter::new(sink, &[Type::INT4, Type::TEXT]);
     pin_mut!(writer);
     writer.as_mut().write(&[&1i32, &"foobar"]).await.unwrap();
-    writer.as_mut().write(&[&2i32, &None::<&str>]).await.unwrap();
+    writer
+        .as_mut()
+        .write(&[&2i32, &None::<&str>])
+        .await
+        .unwrap();
     writer.finish().await.unwrap();
 
     let rows = client
@@ -50,12 +57,19 @@ async fn write_many_rows() {
         .await
         .unwrap();
 
-    let sink = client.copy_in("COPY foo (id, bar) FROM STDIN BINARY", &[]).await.unwrap();
+    let sink = client
+        .copy_in("COPY foo (id, bar) FROM STDIN BINARY", &[])
+        .await
+        .unwrap();
     let writer = BinaryCopyInWriter::new(sink, &[Type::INT4, Type::TEXT]);
     pin_mut!(writer);
 
     for i in 0..10_000i32 {
-        writer.as_mut().write(&[&i, &format!("the value for {}", i)]).await.unwrap();
+        writer
+            .as_mut()
+            .write(&[&i, &format!("the value for {}", i)])
+            .await
+            .unwrap();
     }
 
     writer.finish().await.unwrap();
@@ -79,12 +93,19 @@ async fn write_big_rows() {
         .await
         .unwrap();
 
-    let sink = client.copy_in("COPY foo (id, bar) FROM STDIN BINARY", &[]).await.unwrap();
+    let sink = client
+        .copy_in("COPY foo (id, bar) FROM STDIN BINARY", &[])
+        .await
+        .unwrap();
     let writer = BinaryCopyInWriter::new(sink, &[Type::INT4, Type::BYTEA]);
     pin_mut!(writer);
 
     for i in 0..2i32 {
-        writer.as_mut().write(&[&i, &vec![i as u8; 128 * 1024]]).await.unwrap();
+        writer
+            .as_mut()
+            .write(&[&i, &vec![i as u8; 128 * 1024]])
+            .await
+            .unwrap();
     }
 
     writer.finish().await.unwrap();
@@ -108,13 +129,19 @@ async fn read_basic() {
             "
             CREATE TEMPORARY TABLE foo (id INT, bar TEXT);
             INSERT INTO foo (id, bar) VALUES (1, 'foobar'), (2, NULL);
-            "
+            ",
         )
         .await
         .unwrap();
 
-    let stream = client.copy_out("COPY foo (id, bar) TO STDIN BINARY", &[]).await.unwrap();
-    let rows = BinaryCopyOutStream::new(&[Type::INT4, Type::TEXT], stream).try_collect::<Vec<_>>().await.unwrap();
+    let stream = client
+        .copy_out("COPY foo (id, bar) TO STDIN BINARY", &[])
+        .await
+        .unwrap();
+    let rows = BinaryCopyOutStream::new(&[Type::INT4, Type::TEXT], stream)
+        .try_collect::<Vec<_>>()
+        .await
+        .unwrap();
     assert_eq!(rows.len(), 2);
 
     assert_eq!(rows[0].get::<i32>(0), 1);
@@ -136,8 +163,14 @@ async fn read_many_rows() {
         .await
         .unwrap();
 
-    let stream = client.copy_out("COPY foo (id, bar) TO STDIN BINARY", &[]).await.unwrap();
-    let rows = BinaryCopyOutStream::new(&[Type::INT4, Type::TEXT], stream).try_collect::<Vec<_>>().await.unwrap();
+    let stream = client
+        .copy_out("COPY foo (id, bar) TO STDIN BINARY", &[])
+        .await
+        .unwrap();
+    let rows = BinaryCopyOutStream::new(&[Type::INT4, Type::TEXT], stream)
+        .try_collect::<Vec<_>>()
+        .await
+        .unwrap();
     assert_eq!(rows.len(), 10_000);
 
     for (i, row) in rows.iter().enumerate() {
@@ -155,11 +188,23 @@ async fn read_big_rows() {
         .await
         .unwrap();
     for i in 0..2i32 {
-        client.execute("INSERT INTO foo (id, bar) VALUES ($1, $2)", &[&i, &vec![i as u8; 128 * 1024]]).await.unwrap();
+        client
+            .execute(
+                "INSERT INTO foo (id, bar) VALUES ($1, $2)",
+                &[&i, &vec![i as u8; 128 * 1024]],
+            )
+            .await
+            .unwrap();
     }
 
-    let stream = client.copy_out("COPY foo (id, bar) TO STDIN BINARY", &[]).await.unwrap();
-    let rows = BinaryCopyOutStream::new(&[Type::INT4, Type::BYTEA], stream).try_collect::<Vec<_>>().await.unwrap();
+    let stream = client
+        .copy_out("COPY foo (id, bar) TO STDIN BINARY", &[])
+        .await
+        .unwrap();
+    let rows = BinaryCopyOutStream::new(&[Type::INT4, Type::BYTEA], stream)
+        .try_collect::<Vec<_>>()
+        .await
+        .unwrap();
     assert_eq!(rows.len(), 2);
 
     for (i, row) in rows.iter().enumerate() {