Skip to content

Commit

Permalink
fix: don't derive Hash on types containing recon keys
Browse files Browse the repository at this point in the history
this probalby doesn't hurt to include, but we're particular about using our AHash so we'll leave it explicit for now
  • Loading branch information
dav1do committed Aug 9, 2024
1 parent dff5ab9 commit f1ab99a
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions recon/src/recon.rs
Original file line number Diff line number Diff line change
Expand Up @@ -404,7 +404,7 @@ where
}

/// A key value pair to store
#[derive(Clone, Debug, PartialEq, Eq, Hash)]
#[derive(Clone, Debug, PartialEq, Eq)]
pub struct ReconItem<K>
where
K: Key,
Expand All @@ -429,7 +429,7 @@ where
}

/// Represents reasons the store is unwilling to persist keys and values
#[derive(Clone, Debug, PartialEq, Eq, Hash)]
#[derive(Clone, Debug, PartialEq, Eq)]
pub enum InvalidItem<K: Key> {
/// The key or event data could not be parsed
InvalidFormat {
Expand All @@ -443,7 +443,7 @@ pub enum InvalidItem<K: Key> {
},
}

#[derive(Clone, Debug, PartialEq, Eq, Hash)]
#[derive(Clone, Debug, PartialEq, Eq)]
/// Represents items the store may be able to persist in the future
pub struct PendingItem<K>
where
Expand All @@ -464,7 +464,7 @@ impl<K: Key> PendingItem<K> {
}

/// The result of an insert operation.
#[derive(Clone, Debug, PartialEq, Eq, Hash, Default)]
#[derive(Clone, Debug, PartialEq, Eq, Default)]
pub struct InsertBatch<K>
where
K: Key,
Expand Down

0 comments on commit f1ab99a

Please sign in to comment.