Skip to content

Commit

Permalink
Merge pull request #29395 from vespa-engine/toregge/fix-format-strings
Browse files Browse the repository at this point in the history
Fix format strings.
  • Loading branch information
baldersheim committed Nov 20, 2023
2 parents 2e8c96d + 343fc05 commit d998b27
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
#include <vespa/document/bucket/fixed_bucket_spaces.h>
#include <vespa/vespalib/util/backtrace.h>
#include <cassert>
#include <cinttypes>

#include <vespa/log/log.h>
LOG_SETUP(".distributor.distributor_bucket_space_repo");
Expand Down Expand Up @@ -36,7 +37,7 @@ DistributorBucketSpaceRepo::get(BucketSpace bucketSpace)
{
auto itr = _map.find(bucketSpace);
if (itr == _map.end()) [[unlikely]] {
LOG(error, "Bucket space %zu does not have a valid mapping. %s", bucketSpace.getId(), vespalib::getStackTrace(0).c_str());
LOG(error, "Bucket space %" PRIu64 " does not have a valid mapping. %s", bucketSpace.getId(), vespalib::getStackTrace(0).c_str());
abort();
}
return *itr->second;
Expand All @@ -47,7 +48,7 @@ DistributorBucketSpaceRepo::get(BucketSpace bucketSpace) const
{
auto itr = _map.find(bucketSpace);
if (itr == _map.end()) [[unlikely]] {
LOG(error, "Bucket space %zu does not have a valid mapping. %s", bucketSpace.getId(), vespalib::getStackTrace(0).c_str());
LOG(error, "Bucket space %" PRIu64 " does not have a valid mapping. %s", bucketSpace.getId(), vespalib::getStackTrace(0).c_str());
abort();
}
return *itr->second;
Expand Down
2 changes: 1 addition & 1 deletion storage/src/vespa/storage/persistence/asynchandler.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -319,7 +319,7 @@ AsyncHandler::handle_delete_bucket_throttling(api::DeleteBucketCommand& cmd, Mes
LOG(spam, "%s: completed removeByGidAsync operation", bucket.toString().c_str());
// Nothing else clever to do here. Throttle token and deleteBucket dispatch refs dropped implicitly.
});
LOG(spam, "%s: about to invoke removeByGidAsync(%s, %s, %zu)", cmd.getBucket().toString().c_str(),
LOG(spam, "%s: about to invoke removeByGidAsync(%s, %s, %" PRIu64 ")", cmd.getBucket().toString().c_str(),
vespalib::string(meta->getDocumentType()).c_str(), meta->getGid().toString().c_str(), meta->getTimestamp().getValue());
_spi.removeByGidAsync(spi_bucket, std::move(to_remove), std::make_unique<ResultTaskOperationDone>(_sequencedExecutor, cmd.getBucketId(), std::move(task)));
}
Expand Down

0 comments on commit d998b27

Please sign in to comment.