)]}'
{"trove/tests/api/replication.py":[{"author":{"_account_id":8214,"name":"Auston McReynolds","email":"amcreynolds@ebaysf.com","username":"amcreynolds"},"change_message_id":"943ac09c2db388cb17f7d8d90bf6ef22864a28e9","unresolved":false,"context_lines":[{"line_number":208,"context_line":"        if CONFIG.fake_mode:"},{"line_number":209,"context_line":"            raise SkipTest(\"Test not_read_only not supported in fake mode\")"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"        # wait until replica is no more read only"},{"line_number":212,"context_line":"        def check_not_read_only():"},{"line_number":213,"context_line":"            cmd \u003d \"mysql -BNq -e \\\\\\\u0027select @@read_only\\\\\\\u0027\""},{"line_number":214,"context_line":"            server \u003d create_server_connection(slave_instance.id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_274ca676","line":211,"updated":"2014-11-18 19:29:51.000000000","message":"since you\u0027re already going to need to introduce @time_out (see below), can you fix this to \"wait until replica is no longer read only\"?","commit_id":"c365cd9bff31b9d4e1efe992fbf9dd4d2444c503"},{"author":{"_account_id":8214,"name":"Auston McReynolds","email":"amcreynolds@ebaysf.com","username":"amcreynolds"},"change_message_id":"943ac09c2db388cb17f7d8d90bf6ef22864a28e9","unresolved":false,"context_lines":[{"line_number":217,"context_line":"                return False"},{"line_number":218,"context_line":"            else:"},{"line_number":219,"context_line":"                return True"},{"line_number":220,"context_line":"        poll_until(check_not_read_only)"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    @test(depends_on\u003d[test_detach_replica])"},{"line_number":223,"context_line":"    def test_slave_user_removed(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_c735a2dc","line":220,"updated":"2014-11-18 19:29:51.000000000","message":"since you\u0027ve introduced polling, you want to also introduce @time_out.","commit_id":"c365cd9bff31b9d4e1efe992fbf9dd4d2444c503"},{"author":{"_account_id":10295,"name":"mariam john","email":"mariamj@us.ibm.com","username":"mariamj"},"change_message_id":"52211f7177e4cbf17a8551f6af9e40b62208240d","unresolved":false,"context_lines":[{"line_number":220,"context_line":"                return True"},{"line_number":221,"context_line":"        poll_until(check_not_read_only)"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"    @test(depends_on\u003d[test_detach_replica])"},{"line_number":224,"context_line":"    def test_slave_user_removed(self):"},{"line_number":225,"context_line":"        if CONFIG.fake_mode:"},{"line_number":226,"context_line":"            raise SkipTest(\"Test not_read_only not supported in fake mode\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_c06055a7","line":223,"updated":"2014-11-25 17:52:20.000000000","message":"Riddhi, Looks like this function is also missing \u0027@time_out\u0027. Do you want to include this change in your patch or I can open a new bug. Thanks.","commit_id":"5c66bbc34b774250ea12bdca876f6789ad8b00b9"}]}