Skip to content

Commit

Permalink
storage/segment_appender: missing parameters in vassert messages
Browse files Browse the repository at this point in the history
(cherry picked from commit 7c60a4d)
  • Loading branch information
andijcr authored and vbotbuildovich committed Nov 29, 2023
1 parent 26a1754 commit d813188
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion src/v/storage/segment_appender.cc
Original file line number Diff line number Diff line change
Expand Up @@ -297,6 +297,7 @@ ss::future<> segment_appender::truncate(size_t n) {
n <= file_byte_offset(),
"Cannot ask to truncate at:{} which is more bytes than we have:{} - {}",
file_byte_offset(),
n,
*this);
return hard_flush()
.then([this, n] { return do_truncation(n); })
Expand Down Expand Up @@ -663,7 +664,10 @@ bool segment_appender::inflight_write::try_merge(
// writes should form a contiguous series of writes and we only check
// the last write for merging.
vassert(
committed_offset == pco, "in try_merge writes didn't touch: {} {}");
committed_offset == pco,
"in try_merge writes didn't touch: {} {}",
committed_offset,
pco);

// the lhs write cannot be full since then how could the next write
// share its chunk: it must use a new chunk
Expand Down

0 comments on commit d813188

Please sign in to comment.