)]}'
{"oslo_config/types.py":[{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"4d6ab6feb8660c6283f1869480db840847b34d90","unresolved":true,"context_lines":[{"line_number":630,"context_line":""},{"line_number":631,"context_line":"        if key_value_separator \u003d\u003d \u0027,\u0027:"},{"line_number":632,"context_line":"            raise TypeError(\u0027key_value_separator should not be \\\u0027,\\\u0027\u0027)"},{"line_number":633,"context_line":""},{"line_number":634,"context_line":"        self.value_type \u003d value_type"},{"line_number":635,"context_line":"        self.bounds \u003d bounds"},{"line_number":636,"context_line":"        self.key_value_separator \u003d key_value_separator"}],"source_content_type":"text/x-python","patch_set":1,"id":"6ab4e316_12cf1df5","line":633,"updated":"2024-06-24 12:58:12.000000000","message":"Actually I have read again the modification and I\u0027m not sure now about something. So you add a new parameter that is good to have the possibility to choice another separator. In the commit message you give an example with a comma but here you raise an exception if we use the comma as a separator. Why?","commit_id":"d193c657441f24506f75b9ce1248c8fdebd295f3"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c253bc0c545a3de7eb160ca2937a8f6bceb268c3","unresolved":true,"context_lines":[{"line_number":630,"context_line":""},{"line_number":631,"context_line":"        if key_value_separator \u003d\u003d \u0027,\u0027:"},{"line_number":632,"context_line":"            raise TypeError(\u0027key_value_separator should not be \\\u0027,\\\u0027\u0027)"},{"line_number":633,"context_line":""},{"line_number":634,"context_line":"        self.value_type \u003d value_type"},{"line_number":635,"context_line":"        self.bounds \u003d bounds"},{"line_number":636,"context_line":"        self.key_value_separator \u003d key_value_separator"}],"source_content_type":"text/x-python","patch_set":1,"id":"e992417d_d10d62db","line":633,"in_reply_to":"6ab4e316_12cf1df5","updated":"2024-06-24 13:43:08.000000000","message":"The comma is not a separator between key and value but it\u0027s the separator between key-value pairs. In `disk_cachemode\u003dkey1\u003dvalue1,key2\u003dvalue2` example, separator is \u0027\u003d\u0027.","commit_id":"d193c657441f24506f75b9ce1248c8fdebd295f3"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"1c0a888a3a8fca27b0d2ed34a0186e4683c9139d","unresolved":true,"context_lines":[{"line_number":630,"context_line":""},{"line_number":631,"context_line":"        if key_value_separator \u003d\u003d \u0027,\u0027:"},{"line_number":632,"context_line":"            raise TypeError(\u0027key_value_separator should not be \\\u0027,\\\u0027\u0027)"},{"line_number":633,"context_line":""},{"line_number":634,"context_line":"        self.value_type \u003d value_type"},{"line_number":635,"context_line":"        self.bounds \u003d bounds"},{"line_number":636,"context_line":"        self.key_value_separator \u003d key_value_separator"}],"source_content_type":"text/x-python","patch_set":1,"id":"257fde99_3432dbb4","line":633,"in_reply_to":"e992417d_d10d62db","updated":"2024-06-25 08:31:17.000000000","message":"Yes sorry I have misunderstood.","commit_id":"d193c657441f24506f75b9ce1248c8fdebd295f3"}]}