)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a962dc8a14c5875a222e58f7961625c8a046b5f4","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     niuke \u003cniuke19970315@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-07-27 17:48:39 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"remove unicode from code"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I20d0f4c1091575d2b3b10d9331f3415dcb009055"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"f6487a6a_424846c2","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":24},"updated":"2022-07-27 10:49:41.000000000","message":"Please update this to something like:\n\n  Remove unnecessary explicit unicode literal strings\n\n(it\u0027s a bit more precise about what your patch does)","commit_id":"98e157e28455132b9917fb13001ef4653a87ea2e"},{"author":{"_account_id":35058,"name":"Ke Niu","email":"niuke@inspur.com","username":"niuke"},"change_message_id":"b4786acba40d0033ea1cada9da5fb66fd03837fe","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     niuke \u003cniuke19970315@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-07-27 17:48:39 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"remove unicode from code"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I20d0f4c1091575d2b3b10d9331f3415dcb009055"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"6a71a6c3_02117dc1","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":24},"in_reply_to":"f6487a6a_424846c2","updated":"2022-08-30 06:41:44.000000000","message":"Done","commit_id":"98e157e28455132b9917fb13001ef4653a87ea2e"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a962dc8a14c5875a222e58f7961625c8a046b5f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"beba2357_72c1ed4a","updated":"2022-07-27 10:49:41.000000000","message":"This duplicates https://review.opendev.org/c/openstack/cinder-specs/+/769275 , but that patch has been sitting with no response for over a year.  Please make the correction to the commit message suggested inline, and push a new patch set.  (This note is mostly to remind me to abandon that other patch once yours has been accepted.)","commit_id":"98e157e28455132b9917fb13001ef4653a87ea2e"},{"author":{"_account_id":35058,"name":"Ke Niu","email":"niuke@inspur.com","username":"niuke"},"change_message_id":"b4786acba40d0033ea1cada9da5fb66fd03837fe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"39c6151c_48c0d4cf","in_reply_to":"beba2357_72c1ed4a","updated":"2022-08-30 06:41:44.000000000","message":"Resolved","commit_id":"98e157e28455132b9917fb13001ef4653a87ea2e"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"5009f58d0e96f207af16d548cd42ace4d090b3df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"43e32506_c585e128","updated":"2022-08-10 23:40:29.000000000","message":"I\u0027m OK with this as long as we abandon the duplicate","commit_id":"0423ba499151decf96ab586396202dffc623a8b4"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ebcf11f967e470af47d1c6f640d1b676eac013c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dc866ff2_39600bb4","updated":"2025-12-12 04:13:03.000000000","message":"Sorry this has been sitting so long!  My concerns from July 2022 have been addressed.","commit_id":"0423ba499151decf96ab586396202dffc623a8b4"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"7f1d391cc5f3e78a64fbc0d41d57d2704e4d22cd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bdc6f784_2c38e744","updated":"2025-12-05 15:24:12.000000000","message":"The change makes sense to me if they are truly unnecessary, but it\u0027s too weird that some of these values were tagged and others were not.  I am worried it\u0027s going to cause some subtle issues - such as string comparisons that won\u0027t succeed due to encoding or something.\n\nThat said, I\u0027m guessing you checked that these values aren\u0027t likely to cause problems by removing them from being unicode?","commit_id":"0423ba499151decf96ab586396202dffc623a8b4"},{"author":{"_account_id":35058,"name":"Ke Niu","email":"niuke@inspur.com","username":"niuke"},"change_message_id":"b4786acba40d0033ea1cada9da5fb66fd03837fe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a8252c8d_ce063d1a","in_reply_to":"43e32506_c585e128","updated":"2022-08-30 06:41:44.000000000","message":"Thank you very much for your guidance.","commit_id":"0423ba499151decf96ab586396202dffc623a8b4"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"265a81da662ab3779d6c3aa55617a51b2125fde2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"32387ca2_d4045db1","in_reply_to":"b80f6c93_71dc2e43","updated":"2025-12-12 16:52:04.000000000","message":"Thanks, Brian.","commit_id":"0423ba499151decf96ab586396202dffc623a8b4"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ebcf11f967e470af47d1c6f640d1b676eac013c4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b80f6c93_71dc2e43","in_reply_to":"bdc6f784_2c38e744","updated":"2025-12-12 04:13:03.000000000","message":"This is good to ask, but we did this in cinder with change I5c4b0eb24ecade37c22e7777640466116a893a89 about 4 years ago; the doc/source/conf.py had the same weird mix of plain strings and u\u0027\u0027 strings, they were all changed to plain strings and everything has been fine.  So it should be fine to do it here, too.","commit_id":"0423ba499151decf96ab586396202dffc623a8b4"}]}