Deserialization failure will swallow the query result

[ENJATZ]

Describe the bug

Having a custom deserialization for one of the fields in the expected clickhouse::Row will swallow the returned rows if the deserialization is failing.

Steps to reproduce

1. Have a struct with a field that has custom serialization/deserialization

#[derive(Debug, Clone, serde::Deserialize, serde::Serialize, clickhouse::Row)]
pub struct Event {
    pub nonce: String,
    pub name: String,
    pub source_serial_number: Option<String>,
    pub source_device_id: ch::ChIdOption<model::pg::device::Id>,
}

2. A simple query to to get the rows:

pub async fn get_by_id(
    client: &clickhouse::Client,
    id: &common_database::model::ch::event::Id,
) -> Result<Event, clickhouse::error::Error> {
    client
        .query("SELECT ?fields FROM event WHERE id = ?")
        .bind(id)
        .fetch_one::<Event>()
        .await
}

Expected behaviour

In this case, the targeted event, has a NULL value for source_device_id => which will fall under the case of clickhouse::serde::uuid::option::deserialize(deserializer) or even clickhouse::serde::uuid::deserialize(deserializer) (same result), in this case the uuid::deserialize should return None, but it it throwing an Err(NotEnoughData). Not handling the error, will just swallow the returned row.

Code example

ChIdOption deserialized with:

pub fn deserialize<'de, D, T>(deserializer: D) -> Result<Option<T>, D::Error>
            where
                D: serde::Deserializer<'de>,
                T: crate::DbId,
            {
                clickhouse::serde::uuid::option::deserialize(deserializer)
                    .map(|v| v.map(T::from_unsafe))
            }