)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"a7565b1b001e862695902dd7a9b54fc8da27c007","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Tim Burke \u003ctim.burke@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-05-22 04:05:08 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"tests: Speed up statsd test_methods_are_no_ops_when_not_enabled"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I0a27efe897b4e8ce2c21da1a3603a2a77c02eb69"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"d017da24_477a6251","line":7,"updated":"2025-05-22 09:10:59.000000000","message":"yup, the tearDown is waiting four seconds before timing out listening for something that won\u0027t happen","commit_id":"aa5bc019821b6322245f0ff0d80b5ceafa587c45"}],"test/unit/common/test_statsd_client.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"a7565b1b001e862695902dd7a9b54fc8da27c007","unresolved":true,"context_lines":[{"line_number":708,"context_line":"    Tests here use get_statsd_client to make a StatsdClient."},{"line_number":709,"context_line":"    \"\"\""},{"line_number":710,"context_line":"    def test_methods_are_no_ops_when_not_enabled(self):"},{"line_number":711,"context_line":"        # *Don\u0027t* use self.client -- we want tearDown to create it"},{"line_number":712,"context_line":"        client \u003d get_statsd_client({"},{"line_number":713,"context_line":"            # No \"log_statsd_host\" means \"disabled\""},{"line_number":714,"context_line":"            \u0027log_statsd_port\u0027: str(self.port),"}],"source_content_type":"text/x-python","patch_set":2,"id":"8d691e18_c7130962","line":711,"range":{"start_line":711,"start_character":45,"end_line":711,"end_character":54},"updated":"2025-05-22 09:10:59.000000000","message":"makes me wonder if tearDown should always create a fresh client\n\nBut this is a sensible change nevertheless. Thank you for mac-land! Although I\u0027d not yet noticed the lag.","commit_id":"aa5bc019821b6322245f0ff0d80b5ceafa587c45"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"0a8b6cfe327b773e5b0834a1e7fca0fd5782052e","unresolved":true,"context_lines":[{"line_number":708,"context_line":"    Tests here use get_statsd_client to make a StatsdClient."},{"line_number":709,"context_line":"    \"\"\""},{"line_number":710,"context_line":"    def test_methods_are_no_ops_when_not_enabled(self):"},{"line_number":711,"context_line":"        # *Don\u0027t* use self.client -- we want tearDown to create it"},{"line_number":712,"context_line":"        client \u003d get_statsd_client({"},{"line_number":713,"context_line":"            # No \"log_statsd_host\" means \"disabled\""},{"line_number":714,"context_line":"            \u0027log_statsd_port\u0027: str(self.port),"}],"source_content_type":"text/x-python","patch_set":2,"id":"e1a74b73_9fc3fa98","line":711,"range":{"start_line":711,"start_character":45,"end_line":711,"end_character":54},"in_reply_to":"8d691e18_c7130962","updated":"2025-05-22 15:53:35.000000000","message":"\u003e Although I\u0027d not yet noticed the lag.\n\nMe neither, really. After doing the parent patch, I decided to throw a `--durations\u003d10` on my pytest run and this was the longest-running test.\n```\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d slowest 10 durations \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n4.01s call     test/unit/common/test_statsd_client.py::TestGetStatsdClientOutput::test_methods_are_no_ops_when_not_enabled\n3.03s call     test/unit/proxy/test_server.py::TestProxyServer::test_valid_api_version\n3.03s call     test/unit/proxy/test_mem_server.py::TestProxyServer::test_valid_api_version\n2.50s call     test/unit/common/middleware/test_copy.py::TestServerSideCopyHeartbeat::test_copy_with_heartbeat_success\n2.50s call     test/unit/common/middleware/test_copy.py::TestServerSideCopyHeartbeat::test_copy_with_heartbeat_failure\n1.55s call     test/unit/proxy/test_sysmeta.py::TestObjectSysmeta::test_transient_sysmeta_replaced_by_PUT_or_POST\n1.55s call     test/unit/proxy/test_sysmeta.py::TestObjectSysmeta::test_sysmeta_not_updated_by_POST\n1.54s call     test/unit/common/test_manager.py::TestServer::test_wait\n1.52s call     test/unit/proxy/test_server.py::TestProxyServer::test_node_timing\n1.52s call     test/unit/proxy/test_mem_server.py::TestProxyServer::test_node_timing\n```","commit_id":"aa5bc019821b6322245f0ff0d80b5ceafa587c45"}]}
