remove unnecessary impls

Author: marvin-j97

src/table/index_block/block_handle.rs

@@ -17,7 +17,7 @@ use std::io::{Cursor, Seek};
 use varint_rs::{VarintReader, VarintWriter};
 
 /// Points to a block on file
-#[derive(Copy, Clone, Debug, Default, Eq)]
+#[derive(Copy, Clone, Debug, Default)]
 pub struct BlockHandle {
     /// Position of block in file
     offset: BlockOffset,
@@ -43,24 +43,6 @@ impl BlockHandle {
     }
 }
 
-impl PartialEq for BlockHandle {
-    fn eq(&self, other: &Self) -> bool {
-        self.offset == other.offset
-    }
-}
-
-impl Ord for BlockHandle {
-    fn cmp(&self, other: &Self) -> std::cmp::Ordering {
-        self.offset.cmp(&other.offset)
-    }
-}
-
-impl PartialOrd for BlockHandle {
-    fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> {
-        Some(self.cmp(other))
-    }
-}
-
 impl Encode for BlockHandle {
     fn encode_into<W: std::io::Write>(&self, writer: &mut W) -> Result<(), crate::Error> {
         writer.write_u64_varint(*self.offset)?;
@@ -85,7 +67,7 @@ impl Decode for BlockHandle {
 }
 
 /// Points to a block on file
-#[derive(Clone, Debug, Eq)]
+#[derive(Clone, Debug)]
 pub struct KeyedBlockHandle {
     /// Key of last item in block
     end_key: UserKey,
@@ -138,24 +120,13 @@ impl KeyedBlockHandle {
     }
 }
 
+#[cfg(test)]
 impl PartialEq for KeyedBlockHandle {
     fn eq(&self, other: &Self) -> bool {
         self.offset() == other.offset()
     }
 }
 
-impl Ord for KeyedBlockHandle {
-    fn cmp(&self, other: &Self) -> std::cmp::Ordering {
-        self.offset().cmp(&other.offset())
-    }
-}
-
-impl PartialOrd for KeyedBlockHandle {
-    fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> {
-        Some(self.cmp(other))
-    }
-}
-
 impl Encodable<BlockOffset> for KeyedBlockHandle {
     fn encode_full_into<W: std::io::Write>(
         &self,

src/table/regions.rs

@@ -30,7 +30,7 @@ fn toc_entry_to_handle(entry: &TocEntry) -> BlockHandle {
 /// |--------------|
 /// |   trailer    | <- fixed size
 /// |--------------|
-#[derive(Copy, Clone, Debug, Default, PartialEq, Eq)]
+#[derive(Copy, Clone, Debug, Default)]
 pub struct ParsedRegions {
     pub tli: BlockHandle,
     pub index: Option<BlockHandle>,