)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f6ec09960217a3b2e7c4aeb889112366221d42ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"95f3e8d8_aca76b62","updated":"2024-11-20 01:26:47.000000000","message":"LGTM.","commit_id":"7cb7070df4d73f5e6cc21c54e91a66b4b7c12269"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"cf87ecfc645e745d049fcafa3b60c99d09a104d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"73af0856_405aa06a","updated":"2024-11-20 00:19:18.000000000","message":"Thank you for the review.","commit_id":"7cb7070df4d73f5e6cc21c54e91a66b4b7c12269"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"24230aa7135b155f1c08d78e6fe3b95ba2f5ec2e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"98a78e0e_baa07ab1","updated":"2024-11-19 11:03:26.000000000","message":"Thanks for your patch, almost LGTM.\nI added a question, please kindly confirm it.","commit_id":"7cb7070df4d73f5e6cc21c54e91a66b4b7c12269"}],"tacker/sol_refactored/common/coordinate.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"24230aa7135b155f1c08d78e6fe3b95ba2f5ec2e","unresolved":true,"context_lines":[{"line_number":84,"context_line":"            res_id \u003d res_arg.format(**call_args)"},{"line_number":85,"context_line":"            lock \u003d coord.get_lock(res_id)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"            # NOTE: blocking can be integer and it means timeout period."},{"line_number":88,"context_line":"            # if blocking is True, wait until lock is released."},{"line_number":89,"context_line":"            # NOTE: \u0027with lock\u0027 is not used since it can\u0027t handle"},{"line_number":90,"context_line":"            # lock failed exception well."},{"line_number":91,"context_line":"            if not lock.acquire(blocking\u003dblocking):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ef9a20d0_3902efbe","line":88,"range":{"start_line":87,"start_character":0,"end_line":88,"end_character":63},"updated":"2024-11-19 11:03:26.000000000","message":"Just a question, how does this function work if?\n\n- blocking argument is set to \"1\".\n  -\u003e developer want set timeout period to \"1\", but is it same as \"True\" processing or not?\n\nIf it works the same way, I thought it would be better to add something to the Note for other developers.","commit_id":"7cb7070df4d73f5e6cc21c54e91a66b4b7c12269"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"cf87ecfc645e745d049fcafa3b60c99d09a104d2","unresolved":true,"context_lines":[{"line_number":84,"context_line":"            res_id \u003d res_arg.format(**call_args)"},{"line_number":85,"context_line":"            lock \u003d coord.get_lock(res_id)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"            # NOTE: blocking can be integer and it means timeout period."},{"line_number":88,"context_line":"            # if blocking is True, wait until lock is released."},{"line_number":89,"context_line":"            # NOTE: \u0027with lock\u0027 is not used since it can\u0027t handle"},{"line_number":90,"context_line":"            # lock failed exception well."},{"line_number":91,"context_line":"            if not lock.acquire(blocking\u003dblocking):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fbc57f55_7246f289","line":88,"range":{"start_line":87,"start_character":0,"end_line":88,"end_character":63},"in_reply_to":"ef9a20d0_3902efbe","updated":"2024-11-20 00:19:18.000000000","message":"No, \"blocking\u003d1\" means timeout period is 1 sec.","commit_id":"7cb7070df4d73f5e6cc21c54e91a66b4b7c12269"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f6ec09960217a3b2e7c4aeb889112366221d42ef","unresolved":false,"context_lines":[{"line_number":84,"context_line":"            res_id \u003d res_arg.format(**call_args)"},{"line_number":85,"context_line":"            lock \u003d coord.get_lock(res_id)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"            # NOTE: blocking can be integer and it means timeout period."},{"line_number":88,"context_line":"            # if blocking is True, wait until lock is released."},{"line_number":89,"context_line":"            # NOTE: \u0027with lock\u0027 is not used since it can\u0027t handle"},{"line_number":90,"context_line":"            # lock failed exception well."},{"line_number":91,"context_line":"            if not lock.acquire(blocking\u003dblocking):"}],"source_content_type":"text/x-python","patch_set":1,"id":"85b9804a_7f9d4f4a","line":88,"range":{"start_line":87,"start_character":0,"end_line":88,"end_character":63},"in_reply_to":"fbc57f55_7246f289","updated":"2024-11-20 01:26:47.000000000","message":"Thank you for your answer.\nI understood that the lock.acquire() method judges a bool(True) or integer(1) for the \"blocking\" parameter.","commit_id":"7cb7070df4d73f5e6cc21c54e91a66b4b7c12269"}]}