Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8999 )
Change subject: data_dirs: fix logging message ...................................................................... data_dirs: fix logging message If fs_target_data_dirs_per_tablet is set to be greater than the number of available directories, it will log a message that is dependent on there being a configured metrics entity, which is not always available. This patch avoids the potential nullptr access. This patch also changes the logging to INFO-level instead of WARNING-level, as the message doesn't necessarily indicate a problem. Change-Id: I8135e0ad2411524fcb74b3e349bea7a5828a4237 Reviewed-on: http://gerrit.cloudera.org:8080/8999 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins --- M src/kudu/fs/data_dirs.cc 1 file changed, 8 insertions(+), 4 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8999 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8135e0ad2411524fcb74b3e349bea7a5828a4237 Gerrit-Change-Number: 8999 Gerrit-PatchSet: 3 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins
