You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by da...@apache.org on 2016/05/24 19:30:11 UTC
[1/2] incubator-kudu git commit: KUDU-1459 Updated error sentence
about SSE4.2 support
Repository: incubator-kudu
Updated Branches:
refs/heads/branch-0.9.x 578b51375 -> 2bfa47a9a
KUDU-1459 Updated error sentence about SSE4.2 support
This patch augments the error sentence at initial startup about
the CPU not supporting SSE4.2, with a possible source of the error
when running inside a VM and a pointer in the Quickstart/Troubleshooting
section of the online documentation.
(Virtualbox has a pass-through feature for this)
Change-Id: Icb56a23a528906c04644f36da1386b887328f326
Reviewed-on: http://gerrit.cloudera.org:8080/3152
Reviewed-by: Todd Lipcon <to...@apache.org>
Tested-by: Kudu Jenkins
(cherry picked from commit 0346d029dfc0b369e6eeec5eb5f530f50d73a011)
Reviewed-on: http://gerrit.cloudera.org:8080/3187
Reviewed-by: Jean-Daniel Cryans
Project: http://git-wip-us.apache.org/repos/asf/incubator-kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kudu/commit/4f7581e1
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kudu/tree/4f7581e1
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kudu/diff/4f7581e1
Branch: refs/heads/branch-0.9.x
Commit: 4f7581e1b423e42406a32668ae923625b7913af3
Parents: 578b513
Author: cenku <ce...@gmail.com>
Authored: Fri May 20 15:15:34 2016 +0300
Committer: Jean-Daniel Cryans <jd...@gerrit.cloudera.org>
Committed: Tue May 24 16:16:12 2016 +0000
----------------------------------------------------------------------
docs/quickstart.adoc | 4 ++++
src/kudu/util/init.cc | 3 ++-
2 files changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/4f7581e1/docs/quickstart.adoc
----------------------------------------------------------------------
diff --git a/docs/quickstart.adoc b/docs/quickstart.adoc
index 06f7f8f..13539c2 100644
--- a/docs/quickstart.adoc
+++ b/docs/quickstart.adoc
@@ -263,6 +263,10 @@ Value: 192.168.56.100
contain references to the previous VM's SSH credentials. Remove any references to
`quickstart.cloudera` from this file.
+==== Failing with lack of SSE4.2 support when running inside VirtualBox
+
+* Running Kudu currently requires a CPU that supports SSE4.2 (Nehalem or later for Intel). To pass through SSE4.2 support into the guest VM, refer to the link:https://www.virtualbox.org/manual/ch09.html#sse412passthrough[VirtualBox documentation]
+
== Next Steps
- link:installation.html[Installing Kudu]
- link:configuration.html[Configuring Kudu]
http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/4f7581e1/src/kudu/util/init.cc
----------------------------------------------------------------------
diff --git a/src/kudu/util/init.cc b/src/kudu/util/init.cc
index 9bae7c2..1942798 100644
--- a/src/kudu/util/init.cc
+++ b/src/kudu/util/init.cc
@@ -30,7 +30,8 @@ namespace kudu {
Status BadCPUStatus(const base::CPU& cpu, const char* instruction_set) {
return Status::NotSupported(strings::Substitute(
"The CPU on this system ($0) does not support the $1 instruction "
- "set which is required for running Kudu.",
+ "set which is required for running Kudu. If you are running inside a VM, "
+ "you may need to enable SSE4.2 pass-through.",
cpu.cpu_brand(), instruction_set));
}
[2/2] incubator-kudu git commit: catalog_manager: move VLOG(1)
messages above Async RPC calls to avoid crash
Posted by da...@apache.org.
catalog_manager: move VLOG(1) messages above Async RPC calls to avoid crash
Several functions in this file used a pattern of sending an Async RPC, followed
by logging that the RPC had been sent. However, the callback for the async RPC
could delete the 'this' instance, which would cause the log message to crash.
This only showed up as flakiness in Java unit tests, because they set --v=1
and these messages were VLOG(1).
Change-Id: Ic6beaf940fe42a0c7a3aa5877b7ef65136af210b
Reviewed-on: http://gerrit.cloudera.org:8080/3181
Reviewed-by: Jean-Daniel Cryans
Tested-by: Kudu Jenkins
(cherry picked from commit 2a34d81f84529bc307dd494dbc2c952b3256aa65)
Reviewed-on: http://gerrit.cloudera.org:8080/3188
Project: http://git-wip-us.apache.org/repos/asf/incubator-kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kudu/commit/2bfa47a9
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kudu/tree/2bfa47a9
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kudu/diff/2bfa47a9
Branch: refs/heads/branch-0.9.x
Commit: 2bfa47a9ac19ae059d5efbe61c8e2ca9c81f6535
Parents: 4f7581e
Author: Todd Lipcon <to...@apache.org>
Authored: Mon May 23 20:58:33 2016 -0700
Committer: Jean-Daniel Cryans <jd...@gerrit.cloudera.org>
Committed: Tue May 24 16:16:23 2016 +0000
----------------------------------------------------------------------
src/kudu/master/catalog_manager.cc | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/2bfa47a9/src/kudu/master/catalog_manager.cc
----------------------------------------------------------------------
diff --git a/src/kudu/master/catalog_manager.cc b/src/kudu/master/catalog_manager.cc
index 4fc1675..8a4df2b 100644
--- a/src/kudu/master/catalog_manager.cc
+++ b/src/kudu/master/catalog_manager.cc
@@ -2168,11 +2168,11 @@ class AsyncCreateReplica : public RetrySpecificTSRpcTask {
}
virtual bool SendRequest(int attempt) OVERRIDE {
- ts_proxy_->CreateTabletAsync(req_, &resp_, &rpc_,
- boost::bind(&AsyncCreateReplica::RpcCallback, this));
VLOG(1) << "Send create tablet request to " << permanent_uuid_ << ":\n"
<< " (attempt " << attempt << "):\n"
<< req_.DebugString();
+ ts_proxy_->CreateTabletAsync(req_, &resp_, &rpc_,
+ boost::bind(&AsyncCreateReplica::RpcCallback, this));
return true;
}
@@ -2255,11 +2255,11 @@ class AsyncDeleteReplica : public RetrySpecificTSRpcTask {
req.set_cas_config_opid_index_less_or_equal(*cas_config_opid_index_less_or_equal_);
}
- ts_proxy_->DeleteTabletAsync(req, &resp_, &rpc_,
- boost::bind(&AsyncDeleteReplica::RpcCallback, this));
VLOG(1) << "Send delete tablet request to " << permanent_uuid_
<< " (attempt " << attempt << "):\n"
<< req.DebugString();
+ ts_proxy_->DeleteTabletAsync(req, &resp_, &rpc_,
+ boost::bind(&AsyncDeleteReplica::RpcCallback, this));
return true;
}
@@ -2344,11 +2344,11 @@ class AsyncAlterTable : public RetryingTSRpcTask {
l.Unlock();
- ts_proxy_->AlterSchemaAsync(req, &resp_, &rpc_,
- boost::bind(&AsyncAlterTable::RpcCallback, this));
VLOG(1) << "Send alter table request to " << permanent_uuid()
<< " (attempt " << attempt << "):\n"
<< req.DebugString();
+ ts_proxy_->AlterSchemaAsync(req, &resp_, &rpc_,
+ boost::bind(&AsyncAlterTable::RpcCallback, this));
return true;
}
@@ -2467,10 +2467,10 @@ bool AsyncAddServerTask::SendRequest(int attempt) {
}
*peer->mutable_last_known_addr() = peer_reg.rpc_addresses(0);
peer->set_member_type(RaftPeerPB::VOTER);
+ VLOG(1) << "Sending AddServer ChangeConfig request to " << permanent_uuid() << ":\n"
+ << req_.DebugString();
consensus_proxy_->ChangeConfigAsync(req_, &resp_, &rpc_,
boost::bind(&AsyncAddServerTask::RpcCallback, this));
- VLOG(1) << "Sent AddServer ChangeConfig request to " << permanent_uuid() << ":\n"
- << req_.DebugString();
return true;
}