[ https://issues.apache.org/jira/browse/IMPALA-4483?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Tim Armstrong resolved IMPALA-4483. ----------------------------------- Resolution: Fixed Assignee: Jim Apple Fix Version/s: Impala 2.10.0 Fixed by this commit: /request-pool-service.cc: StatsMetric<double>::CreateAndRegister(metrics, RESOLVE_POOL_METRIC_NAME); be/src/scheduling/request-pool-service.h: StatsMetric<double>* resolve_pool_ms_metric_; be/src/statestore/statestore-subscriber.cc: topic_update_interval_metric_ = StatsMetric<double>::CreateAndRegister(metrics, be/src/statestore/statestore-subscriber.cc: topic_update_duration_metric_ = StatsMetric<double>::CreateAndRegister(metrics, be/src/statestore/statestore-subscriber.cc: heartbeat_interval_metric_ = StatsMetric<double>::CreateAndRegister(metrics, be/src/statestore/statestore-subscriber.cc: cb->processing_time_metric = StatsMetric<double>::CreateAndRegister(metrics_, be/src/statestore/statestore-subscriber.h: StatsMetric<double>* processing_time_metric; be/src/statestore/statestore-subscriber.h: StatsMetric<double>* topic_update_interval_metric_; be/src/statestore/statestore-subscriber.h: StatsMetric<double>* topic_update_duration_metric_; be/src/statestore/statestore-subscriber.h: StatsMetric<double>* heartbeat_interval_metric_; be/src/statestore/statestore.cc: StatsMetric<double>::CreateAndRegister(metrics, STATESTORE_UPDATE_DURATION); be/src/statestore/statestore.cc: StatsMetric<double>::CreateAndRegister(metrics, STATESTORE_HEARTBEAT_DURATION); be/src/statestore/statestore.h: StatsMetric<double>* topic_update_duration_metric_; be/src/statestore/statestore.h: StatsMetric<double>* heartbeat_duration_metric_; be/src/util/collection-metrics.h:/// Enum to define which statistic types are available in the StatsMetric be/src/util/collection-metrics.h:class StatsMetric : public Metric { be/src/util/collection-metrics.h: static StatsMetric* CreateAndRegister(MetricGroup* metrics, const std::string& key, be/src/util/collection-metrics.h: return metrics->RegisterMetric(new StatsMetric(MetricDefs::Get(key, arg))); be/src/util/collection-metrics.h: StatsMetric(const TMetricDef& def) : Metric(def), unit_(def.units) { be/src/util/impalad-metrics.cc:StatsMetric<uint64_t, StatsType::MEAN>* be/src/util/impalad-metrics.cc: StatsMetric<uint64_t, StatsType::MEAN>::CreateAndRegister(m, be/src/util/impalad-metrics.h: static StatsMetric<uint64_t, StatsType::MEAN>* IO_MGR_CACHED_FILE_HANDLES_HIT_RATIO; be/src/util/metrics-test.cc:TEST_F(MetricsTest, StatsMetrics) { be/src/util/metrics-test.cc: MetricGroup metrics("StatsMetrics"); be/src/util/metrics-test.cc: StatsMetric<double>* stats_metric = StatsMetric<double>::CreateAndRegister(&metrics, be/src/util/metrics-test.cc: StatsMetric<double>* stats_metric_with_units = be/src/util/metrics-test.cc: StatsMetric<double>::CreateAndRegister(&metrics, "stats_units"); be/src/util/metrics-test.cc:TEST_F(MetricsTest, StatsMetricsSingle) { be/src/util/metrics-test.cc: MetricGroup metrics("StatsMetrics"); be/src/util/metrics-test.cc: StatsMetric<int, StatsType::MAX | StatsType::MEAN>* be/src/util/metrics-test.cc: StatsMetric<int, StatsType::MAX | StatsType::MEAN>::CreateAndRegister(&metrics, be/src/util/metrics-test.cc:TEST_F(MetricsTest, StatsMetricsJson) { be/src/util/metrics-test.cc: MetricGroup metrics("StatsMetrics"); be/src/util/metrics-test.cc: StatsMetric<double>* metric = StatsMetric<double>::CreateAndRegister(&metrics, tarmstrong@tarmstrong-box:~/Impala/incubator-impala$ .,/^C tarmstrong@tarmstrong-box:~/Impala/incubator-impala$ vim bin/run-backend-tests.sh tarmstrong@tarmstrong-box:~/Impala/incubator-impala$ git log bin/run-backend-tests.sh commit 514e04ea4ff3b7dc4225144b869616f2f92e4573 Author: Jim Apple <jbapple-imp...@apache.org> Date: Sat May 20 16:22:01 2017 -0700 Ninja support: use MAKE_CMD in run-backend-tests.sh when possible Without this, buildall.sh -ninja fails to run the backend tests or runs them with the Makefiles that were created when buildall.sh was last run without the -ninja flag. Change-Id: Idb920dd4b08d8ef5fbc0bf1ea1b424a0c544e1db Reviewed-on: http://gerrit.cloudera.org:8080/6942 Reviewed-by: Tim Armstrong <tarmstr...@cloudera.com> Tested-by: Impala Public Jenkins > run-backend-tests.sh does not work with ninja > --------------------------------------------- > > Key: IMPALA-4483 > URL: https://issues.apache.org/jira/browse/IMPALA-4483 > Project: IMPALA > Issue Type: Bug > Components: Infrastructure > Affects Versions: Impala 2.8.0 > Reporter: Jim Apple > Assignee: Jim Apple > Priority: Minor > Labels: newbie > Fix For: Impala 2.10.0 > > > run-backend-tests.hs runs with make, not $MAKE_CMD like other files in bin. > If there is no makefile because ninja was used for the build, then this > script cannot run. Switching it to use ninja, however, isn't a complete > solution, because ninja is so spartan with its output that it provides much > less feedback. -- This message was sent by Atlassian JIRA (v6.4.14#64029)