----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20673/ -----------------------------------------------------------
(Updated April 24, 2014, 10:34 a.m.) Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone. Changes ------- added more to description Repository: mesos-git Description (updated) ------- see summary. without this patch, running 'make check --gtest_filter="MasterTest*"' fails 100% in MasterTest.MetricsInStatsEndpoint. With this patch, this passes 100%. Running MasterTest.MetricsInStatsEndpoint in isolation passes with or without this patch. This leads me to conclude that there's something in the way we setup and tear down master tests that is racy. I think that we are somehow ending up with a gauge in the metrics system that is trying to call a deferred object that references a dangling registrar process reference. My theory right now is that the Gauges struct is being destroyed but the MetricsProcess removal has not run before we hit the endpoint. Diffs ----- src/master/registrar.cpp 8e7760b2abd5f4a80a8a4b1b4b90eefc1136fe66 Diff: https://reviews.apache.org/r/20673/diff/ Testing ------- make check Thanks, Dominic Hamon
