better comments

IvanPleshkov · IvanPleshkov · commit 4ee66c576ae7 · 2026-01-06T14:09:03.000+01:00
diff --git a/lib/segment/src/id_tracker/mutable_id_tracker.rs b/lib/segment/src/id_tracker/mutable_id_tracker.rs
@@ -401,7 +401,7 @@ fn store_mapping_changes(
     mappings_path: &Path,
     changes: &Vec<MappingChange>,
 ) -> OperationResult<()> {
-    // Estimate required buffer size
+    // Estimate required buffer size.
     let mut counting_write = CountingWrite::default();
     write_mapping_changes(&mut counting_write, changes).map_err(|err| {
         OperationError::service_error(format!(
@@ -410,7 +410,7 @@ fn store_mapping_changes(
         ))
     })?;
 
-    // Create a buffer with exact required size
+    // Create a buffer with exact required size.
     let mut writer: Vec<u8> = Vec::default();
     writer
         .try_set_capacity_exact(counting_write.bytes_written() as usize)
@@ -421,21 +421,21 @@ fn store_mapping_changes(
             ))
         })?;
 
-    // Collect all changes into a single buffer
+    // Collect all changes into a single buffer.
     write_mapping_changes(&mut writer, changes).map_err(|err| {
         OperationError::service_error(format!(
             "Failed to collect ID tracker point mappings ({}): {err}",
             mappings_path.display(),
         ))
     })?;
 
-    // Create or open file in append mode to write new changes to the end
+    // Create or open file in append mode to write new changes to the end.
     let file = File::options()
         .create(true)
         .append(true)
         .open(mappings_path)?;
 
-    // Write all changes in one shot
+    // Write all changes in one shot.
     let mut file_writer = BufWriter::new(file);
     file_writer.write_all(&writer).map_err(|err| {
         OperationError::service_error(format!(
@@ -444,11 +444,11 @@ fn store_mapping_changes(
         ))
     })?;
 
-    // Drop the buffer to free memory
+    // Drop the buffer to free memory.
     drop(writer);
 
-    // Explicitly fsync file contents to ensure durability
-    // `BufWriter::into_inner` flushes the buffer as well
+    // Explicitly fsync file contents to ensure durability.
+    // `BufWriter::into_inner` flushes the buffer as well.
     let file = file_writer.into_inner().map_err(|err| {
         OperationError::service_error(format!("Failed to flush ID tracker point mappings: {err}"))
     })?;
