address CR.

apache · Dec 21, 2022 · 5b63bb8 · 5b63bb8
1 parent 437dcab
commit 5b63bb8
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/table_engine/src/partition/rule.rs b/table_engine/src/partition/rule.rs
@@ -14,13 +14,13 @@ define_result!(Error);
 
 /// Partition rule locate partition
 pub trait PartitionRule {
-    /// Locate the partition for each row in `row_group`
+    /// Locate the partition for each row in `row_group`.
     ///
     /// Len of returned value should be equal to the one of rows in `row group`.
-    fn locate_partitions_for_write(row_group: &RowGroup) -> Vec<usize>;
+    fn locate_partitions_for_write(&self, row_group: &RowGroup) -> Vec<usize>;
 
-    /// locate partitions according to `filters`
-    fn locate_partitions_for_read(filters: &[PartitionFilter]) -> Vec<usize>;
+    /// Locate partitions according to `filters`.
+    fn locate_partitions_for_read(&self, filters: &[PartitionFilter]) -> Vec<usize>;
 }
 
 /// Filter using for partition