Github user zuyu commented on a diff in the pull request: https://github.com/apache/incubator-quickstep/pull/362#discussion_r224539118 --- Diff: third_party/src/tmb/tests/message_bus_unittest_common.h --- @@ -205,7 +207,7 @@ class ConnectorThread : public Thread { } protected: - void Run() { + void Run() override { --- End diff -- Add `override` destructors in this file.
---