RUST-852 Move Collection generic trait bounds to implementation (#379)

Author: isabelatkinson

Diff for: src/coll/mod.rs

@@ -3,7 +3,6 @@ pub mod options;
 use std::{fmt::Debug, sync::Arc};
 
 use futures_util::stream::TryStreamExt;
-use serde::{de::DeserializeOwned, Serialize};
 
 use crate::{
     bson::{Bson, Document},
@@ -137,10 +136,7 @@ impl Database {
     ///
     /// This method does not send or receive anything across the wire to the database, so it can be
     /// used repeatedly without incurring any costs from I/O.
-    pub fn collection<T>(&self, name: &str) -> Collection<T>
-    where
-        T: Serialize + DeserializeOwned + Unpin + Debug,
-    {
+    pub fn collection<T>(&self, name: &str) -> Collection<T> {
         Collection::new(self.clone(), name, None)
     }
 
@@ -154,10 +150,7 @@ impl Database {
         &self,
         name: &str,
         options: CollectionOptions,
-    ) -> Collection<T>
-    where
-        T: Serialize + DeserializeOwned + Unpin + Debug,
-    {
+    ) -> Collection<T> {
         Collection::new(self.clone(), name, Some(options))
     }
 

Diff for: src/db/mod.rs

@@ -4,7 +4,7 @@ mod test;
 
 use std::fmt::Debug;
 
-use serde::{de::DeserializeOwned, Deserialize, Serialize};
+use serde::{de::DeserializeOwned, Deserialize};
 
 use self::options::FindAndModifyOptions;
 use crate::{
@@ -27,7 +27,7 @@ use crate::{
 
 pub(crate) struct FindAndModify<T = Document>
 where
-    T: Serialize + DeserializeOwned + Unpin + Debug,
+    T: DeserializeOwned,
 {
     ns: Namespace,
     query: Document,
@@ -37,7 +37,7 @@ where
 
 impl<T> FindAndModify<T>
 where
-    T: Serialize + DeserializeOwned + Unpin + Debug,
+    T: DeserializeOwned,
 {
     pub fn with_delete(
         ns: Namespace,
@@ -97,7 +97,7 @@ where
 
 impl<T> Operation for FindAndModify<T>
 where
-    T: Serialize + DeserializeOwned + Unpin + Debug,
+    T: DeserializeOwned,
 {
     type O = Option<T>;
     const NAME: &'static str = "findAndModify";

Diff for: src/operation/find_and_modify/mod.rs

@@ -60,7 +60,7 @@ pub(crate) use update::Update;
 /// A trait modeling the behavior of a server side operation.
 pub(crate) trait Operation {
     /// The output type of this operation.
-    type O: Debug;
+    type O;
 
     /// The name of the server side command associated with this operation.
     const NAME: &'static str;