You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2016/12/08 01:06:20 UTC
[5/9] mesos git commit: Fixed incorrect warning messages.
Fixed incorrect warning messages.
These warning messages printed the old value of the `FrameworkInfo`
field, not the new (ignored) value.
Review: https://reviews.apache.org/r/54467/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/ca1e7249
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/ca1e7249
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/ca1e7249
Branch: refs/heads/master
Commit: ca1e72499ef877aa6a7942d231f9089fa98da457
Parents: 0dbceaf
Author: Neil Conway <ne...@gmail.com>
Authored: Wed Dec 7 17:04:35 2016 -0800
Committer: Vinod Kone <vi...@gmail.com>
Committed: Wed Dec 7 17:05:46 2016 -0800
----------------------------------------------------------------------
src/master/master.hpp | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/ca1e7249/src/master/master.hpp
----------------------------------------------------------------------
diff --git a/src/master/master.hpp b/src/master/master.hpp
index c0bdb80..d3c20d7 100644
--- a/src/master/master.hpp
+++ b/src/master/master.hpp
@@ -2434,7 +2434,7 @@ struct Framework
// MESOS-703.
if (source.user() != info.user()) {
- LOG(WARNING) << "Cannot update FrameworkInfo.user to '" << info.user()
+ LOG(WARNING) << "Cannot update FrameworkInfo.user to '" << source.user()
<< "' for framework " << id() << ". Check MESOS-703";
}
@@ -2448,12 +2448,12 @@ struct Framework
if (source.checkpoint() != info.checkpoint()) {
LOG(WARNING) << "Cannot update FrameworkInfo.checkpoint to '"
- << stringify(info.checkpoint()) << "' for framework " << id()
- << ". Check MESOS-703";
+ << stringify(source.checkpoint()) << "' for framework "
+ << id() << ". Check MESOS-703";
}
if (source.role() != info.role()) {
- LOG(WARNING) << "Cannot update FrameworkInfo.role to '" << info.role()
+ LOG(WARNING) << "Cannot update FrameworkInfo.role to '" << source.role()
<< "' for framework " << id() << ". Check MESOS-703";
}
@@ -2465,7 +2465,7 @@ struct Framework
if (source.principal() != info.principal()) {
LOG(WARNING) << "Cannot update FrameworkInfo.principal to '"
- << info.principal() << "' for framework " << id()
+ << source.principal() << "' for framework " << id()
<< ". Check MESOS-703";
}