Skip to content

Commit

Permalink
Merge pull request MaterializeInc#21436 from danhhz/persist_warn_batch
Browse files Browse the repository at this point in the history
persist: fix incorrect "dangling blob keys" WARN
  • Loading branch information
danhhz authored Aug 28, 2023
2 parents 003ae64 + 8fc8234 commit c3bf5ba
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/persist-client/src/batch.rs
Original file line number Diff line number Diff line change
Expand Up @@ -177,12 +177,14 @@ where
/// dropped, the data that it represents will have leaked. The caller is
/// responsible for turning this back into a [`Batch`] using
/// [`WriteHandle::batch_from_transmittable_batch`](crate::write::WriteHandle::batch_from_transmittable_batch).
pub fn into_transmittable_batch(self) -> ProtoBatch {
ProtoBatch {
pub fn into_transmittable_batch(mut self) -> ProtoBatch {
let ret = ProtoBatch {
shard_id: self.shard_id.into_proto(),
version: self.version.to_string(),
batch: Some(self.batch.into_proto()),
}
};
self.mark_consumed();
ret
}
}

Expand Down

0 comments on commit c3bf5ba

Please sign in to comment.