Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15055 )
Change subject: [metrics] fix compilation on macOS ...................................................................... Patch Set 1: > Why doesn't this code in metrics.h address this? > > #if defined(__APPLE__) > #define METRIC_DEFINE_gauge_size(entity, name, label, unit, desc, > level, ...) \ > ::kudu::GaugePrototype<size_t> METRIC_##name( \ > ::kudu::MetricPrototype::CtorArgs(#entity, #name, label, unit, > desc, level, ## __VA_ARGS__)) > #define METRIC_DECLARE_gauge_size(name) \ > extern ::kudu::GaugePrototype<size_t> METRIC_##name > #else > #define METRIC_DEFINE_gauge_size METRIC_DEFINE_gauge_uint64 > #define METRIC_DECLARE_gauge_size METRIC_DECLARE_gauge_uint64 > #endif Because on macOS size_t isn't uint64_t. -- To view, visit http://gerrit.cloudera.org:8080/15055 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I8e7109aebec268db1d545e657c991e7791f85210 Gerrit-Change-Number: 15055 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Thu, 16 Jan 2020 22:38:29 +0000 Gerrit-HasComments: No
