)]}'
{"troveclient/v1/shell.py":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"ea4b2a03a45dcff498d0f570ae6c9d5002b55d15","unresolved":false,"context_lines":[{"line_number":1514,"context_line":"            args.parameter)"},{"line_number":1515,"context_line":"    else:"},{"line_number":1516,"context_line":"        raise exceptions.ValidationError("},{"line_number":1517,"context_line":"            \"The datastore name or ID is required to show the information \""},{"line_number":1518,"context_line":"            \"on a configuration parameter\""},{"line_number":1519,"context_line":"        )"},{"line_number":1520,"context_line":"    _print_object(param)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a8bc5a2_981b511b","line":1517,"updated":"2016-09-23 03:21:40.000000000","message":"need i18n","commit_id":"79bdaa935278f853f8eb44c06ea979196f5c104e"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"05b4a953d465147c1449b62cff4f21b7f56ec272","unresolved":false,"context_lines":[{"line_number":1514,"context_line":"            args.parameter)"},{"line_number":1515,"context_line":"    else:"},{"line_number":1516,"context_line":"        raise exceptions.ValidationError("},{"line_number":1517,"context_line":"            \"The datastore name or ID is required to show the information \""},{"line_number":1518,"context_line":"            \"on a configuration parameter\""},{"line_number":1519,"context_line":"        )"},{"line_number":1520,"context_line":"    _print_object(param)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_9d106daa","line":1517,"in_reply_to":"5a8bc5a2_981b511b","updated":"2016-10-06 15:11:08.000000000","message":"agreed.  Plus when I run this with a \u0027bad\u0027 ds_ver, the message is still a bit misleading:\n\n  \u003e trove configuration-parameter-show bad_ds_ver good_param\n  ERROR: The datastore name or ID is required to show the information on a configuration parameter\n\nMaybe something like:\n\n  ERROR: The datastore name or ID is required when specifying a datastore version name.","commit_id":"79bdaa935278f853f8eb44c06ea979196f5c104e"},{"author":{"_account_id":13562,"name":"Samuel Matzek","email":"smatzek@us.ibm.com","username":"smatzek"},"change_message_id":"763e6c10018ea07ba8364bfbb2c6de321ac68ee4","unresolved":false,"context_lines":[{"line_number":1594,"context_line":"@utils.service_type(\u0027database\u0027)"},{"line_number":1595,"context_line":"def do_configuration_parameter_show(cs, args):"},{"line_number":1596,"context_line":"    \"\"\"Shows details of a configuration parameter.\"\"\""},{"line_number":1597,"context_line":"    if utils.is_uuid_like(args.datastore_version):"},{"line_number":1598,"context_line":"        param \u003d cs.configuration_parameters.get_parameter_by_version("},{"line_number":1599,"context_line":"            args.datastore_version,"},{"line_number":1600,"context_line":"            args.parameter)"}],"source_content_type":"text/x-python","patch_set":7,"id":"3f4b6375_7b2b7194","line":1597,"updated":"2017-11-02 17:18:31.000000000","message":"Why did you switch the order of the existing if / elif ?  Am I missing something?","commit_id":"063360cfe2f6f4c7683d133ff6ff086f72b4cf2c"},{"author":{"_account_id":14257,"name":"Mahito Ogura","email":"m.ogura@ntt.com","username":"Mahito"},"change_message_id":"d0dab909a5ca98b9a0e868a473e968e0869168d5","unresolved":false,"context_lines":[{"line_number":1594,"context_line":"@utils.service_type(\u0027database\u0027)"},{"line_number":1595,"context_line":"def do_configuration_parameter_show(cs, args):"},{"line_number":1596,"context_line":"    \"\"\"Shows details of a configuration parameter.\"\"\""},{"line_number":1597,"context_line":"    if utils.is_uuid_like(args.datastore_version):"},{"line_number":1598,"context_line":"        param \u003d cs.configuration_parameters.get_parameter_by_version("},{"line_number":1599,"context_line":"            args.datastore_version,"},{"line_number":1600,"context_line":"            args.parameter)"}],"source_content_type":"text/x-python","patch_set":7,"id":"3f4b6375_e30f8934","line":1597,"in_reply_to":"3f4b6375_7b2b7194","updated":"2017-11-06 01:02:38.000000000","message":"I thought that this is more suitable for the specification\nWhen datastore_version is uuid Because the data is uniquely determined, I thought that datastore is not necessary and changed the order.","commit_id":"063360cfe2f6f4c7683d133ff6ff086f72b4cf2c"}]}