remove unused trait bounds and outdated doc comment

src/collector/top_collector.rs

@@ -15,11 +15,6 @@ use crate::{DocAddress, DocId, SegmentOrdinal};
 /// The REVERSE_ORDER generic parameter controls whether the by-feature order
 /// should be reversed, which is useful for achieving for example largest-first
 /// semantics without having to wrap the feature in a `Reverse`.
-///
-/// WARNING: equality is not what you would expect here.
-/// Two elements are equal if their feature is equal, and regardless of whether `doc`
-/// is equal. This should be perfectly fine for this usage, but let's make sure this
-/// struct is never public.
 #[derive(Clone, Default, Serialize, Deserialize)]
 pub struct ComparableDoc<T, D, const REVERSE_ORDER: bool = false> {
     /// The feature of the document. In practice, this is

src/collector/top_score_collector.rs

@@ -3,7 +3,6 @@ use std::marker::PhantomData;
 use std::sync::Arc;
 
 use columnar::ColumnValues;
-use serde::de::DeserializeOwned;
 use serde::{Deserialize, Serialize};
 
 use super::Collector;
@@ -790,7 +789,7 @@ impl<Score, D, const R: bool> From<TopNComputerDeser<Score, D, R>> for TopNCompu
 impl<Score, D, const R: bool> TopNComputer<Score, D, R>
 where
     Score: PartialOrd + Clone,
-    D: Serialize + DeserializeOwned + Ord + Clone,
+    D: Ord,
 {
     /// Create a new `TopNComputer`.
     /// Internally it will allocate a buffer of size `2 * top_n`.