You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by ad...@apache.org on 2017/09/18 23:08:20 UTC
[3/3] kudu git commit: data_dirs: fix a LOG statement
data_dirs: fix a LOG statement
This is currently logging like so:
No directories available to add to 2e4ba76514974786aa2fabda3ac78ae1's directory group (1 dirs total, 0x60400004e490 full, 0 failed). (error 28)
Change-Id: I6fbf51d3c1a5900808146f0d9c62b9b3a193b7d3
Reviewed-on: http://gerrit.cloudera.org:8080/8091
Reviewed-by: Andrew Wong <aw...@cloudera.com>
Tested-by: Kudu Jenkins
Reviewed-by: David Ribeiro Alves <da...@gmail.com>
Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/23f0b514
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/23f0b514
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/23f0b514
Branch: refs/heads/master
Commit: 23f0b514fa25d03ecd0558d4e531cd1c2c94269c
Parents: c1f78ff
Author: Adar Dembo <ad...@cloudera.com>
Authored: Sun Sep 17 18:39:15 2017 -0700
Committer: Adar Dembo <ad...@cloudera.com>
Committed: Mon Sep 18 23:07:45 2017 +0000
----------------------------------------------------------------------
src/kudu/fs/data_dirs.cc | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kudu/blob/23f0b514/src/kudu/fs/data_dirs.cc
----------------------------------------------------------------------
diff --git a/src/kudu/fs/data_dirs.cc b/src/kudu/fs/data_dirs.cc
index 905b6cd..42fa7f3 100644
--- a/src/kudu/fs/data_dirs.cc
+++ b/src/kudu/fs/data_dirs.cc
@@ -728,7 +728,8 @@ Status DataDirManager::GetNextDataDir(const CreateBlockOptions& opts, DataDir**
}
string dirs_state_str = Substitute("$0 failed", failed_data_dirs_.size());
if (metrics_) {
- dirs_state_str = Substitute("$0 full, $1", metrics_->data_dirs_full.get(), dirs_state_str);
+ dirs_state_str = Substitute("$0 full, $1",
+ metrics_->data_dirs_full->value(), dirs_state_str);
}
return Status::IOError(Substitute("No directories available to add to $0directory group ($1 "
"dirs total, $2).", tablet_id_str, data_dirs_.size(), dirs_state_str),