)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"96269395ca390b09c35ce7accc98665538eaa4b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"00c1ec48_e3b4e79b","updated":"2022-08-30 05:01:22.000000000","message":"I don\u0027t see why this should be done. While the u\u0027 prefix is redundant in python3, it is also not deprecated in any way afaict, so I don\u0027t see a reason to touch all this code.","commit_id":"b04f151d8c4749724d16db893954e6d39f11d003"},{"author":{"_account_id":35058,"name":"Ke Niu","email":"niuke@inspur.com","username":"niuke"},"change_message_id":"02791612fdd0d06de16f63e627bbd61a1e3801a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4e2813fe_469d0f4f","in_reply_to":"00c1ec48_e3b4e79b","updated":"2022-08-30 05:56:17.000000000","message":"All strings are considered as unicode string from Python 3.\n\nThis patch drops the explicit unicode literal (u\u0027...\u0027)\nappearances from the unicode strings.","commit_id":"b04f151d8c4749724d16db893954e6d39f11d003"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2089f6663da34b5f5c8ae92abbac836416880f55","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"91216bdb_198aea36","in_reply_to":"4e2813fe_469d0f4f","updated":"2022-08-30 07:56:35.000000000","message":"This explains *what* you are doing, it does not explain *why* this should be done. To me, there are two options:\n\n1. Don\u0027t do this patch, leave the code as it is, since it is working perfectly well. Maybe clean things up if that code ever needs to be touched for a different reason.\n\n2. Apply your patch.\n\nI prefer 1., I would like to see some argument as to why you prefer 2. Also, as mentioned elsewhere, if 2. is to be done, it should be done consistently across all of opendev, which would require prior discussion with all the teams involved. Submitting such a mass of patches without this preparation isn\u0027t good style and should be avoided in general.","commit_id":"b04f151d8c4749724d16db893954e6d39f11d003"},{"author":{"_account_id":35058,"name":"Ke Niu","email":"niuke@inspur.com","username":"niuke"},"change_message_id":"2932ba0d53c3e11c401bf19fda6b9093c3541b51","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d666ef53_3af5fff2","in_reply_to":"91216bdb_198aea36","updated":"2022-08-30 08:04:17.000000000","message":"Ok, thank you very much for your guidance. I agree with you and I will give up this modification. In addition, I am very sorry for submitting a lot of changes without preparation. I will pay attention to it in the future. Thank you again.","commit_id":"b04f151d8c4749724d16db893954e6d39f11d003"}]}
