Skip to content

Commit

Permalink
[yugabyte#499] docdb: Fixed overflow bug when printing records in dur…
Browse files Browse the repository at this point in the history
…ing a compaction.

Summary: Fixed bug where when printing number of records in and records dropped during a compaction, the number would overflow due to using %d in the print statement instead of %llu

Test Plan: Manually observed log files

Reviewers: jmeehan

Reviewed By: jmeehan

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D15619
  • Loading branch information
parikhdev0 authored and jayant anand committed Mar 8, 2022
1 parent 7dc868e commit 8221592
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/yb/rocksdb/db/compaction_job.cc
Original file line number Diff line number Diff line change
Expand Up @@ -525,7 +525,7 @@ Status CompactionJob::Install(const MutableCFOptions& mutable_cf_options) {
"[%s] compacted to: %s, MB/sec: %.1f rd, %.1f wr, level %d, "
"files in(%d, %d) out(%d) "
"MB in(%.1f, %.1f) out(%.1f), read-write-amplify(%.1f) "
"write-amplify(%.1f) %s, records in: %d, records dropped: %d\n",
"write-amplify(%.1f) %s, records in: %llu, records dropped: %llu\n",
cfd->GetName().c_str(), vstorage->LevelSummary(&tmp),
(stats.bytes_read_non_output_levels + stats.bytes_read_output_level) /
static_cast<double>(stats.micros),
Expand Down

0 comments on commit 8221592

Please sign in to comment.