Hello Adar Dembo, Hannah Nguyen,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/14516
to review the following change.
Change subject: util: fix macOS build of thread.cc
......................................................................
util: fix macOS build of thread.cc
On macOS builds would fail with:
Undefined symbols for architecture x86_64:
"kudu::EasyJson& kudu::EasyJson::operator=<unsigned long>(unsigned long)",
referenced from:
kudu::ThreadMgr::ThreadPathHandler(kudu::WebCallbackRegistry::WebRequest
const&, kudu::WebCallbackRegistry::WebResponse*) const in thread.cc.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Rather than instantiating an EasyJson template for unsigned long, which
would be ambiguous, this replaces the usage of size_t with uint64_t.
Change-Id: Iddb61d9a933a9b76ec73fbfd0a4ef555bc27786d
---
M src/kudu/util/thread.cc
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/16/14516/1
--
To view, visit http://gerrit.cloudera.org:8080/14516
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iddb61d9a933a9b76ec73fbfd0a4ef555bc27786d
Gerrit-Change-Number: 14516
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Hannah Nguyen <[email protected]>
