)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"35f31cc4e6326ac0f897fd872c7a5f642507bea8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c32478f6_9f4b8609","updated":"2022-09-06 18:03:58.000000000","message":"LGTM","commit_id":"7460ae30bf92b5bdf68137bc5de9f7891a33b89b"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"482535666751d6874a06bd2985fc9526a772da9f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2641f0b3_3774068d","updated":"2022-09-06 17:58:59.000000000","message":"LGTM, thanks Stephen.","commit_id":"7460ae30bf92b5bdf68137bc5de9f7891a33b89b"}],"designate/sqlalchemy/session.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"ec5f874c6e81e4a1fd1dc485509be7fedc44a302","unresolved":true,"context_lines":[{"line_number":66,"context_line":"        # LegacyEngineFacade) asap since it\u0027s not compatible with SQLAlchemy"},{"line_number":67,"context_line":"        # 2.0"},{"line_number":68,"context_line":"        conf[\u0027autocommit\u0027] \u003d True"},{"line_number":69,"context_line":"        _FACADES[cache_name] \u003d session.EngineFacade("},{"line_number":70,"context_line":"            connection,"},{"line_number":71,"context_line":"            **conf"},{"line_number":72,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"a2c942dc_0af9773c","line":69,"range":{"start_line":69,"start_character":39,"end_line":69,"end_character":51},"updated":"2022-09-05 18:20:38.000000000","message":"Is EngineFacade just an alias for LegacyEngineFacade? Otherwise I don\u0027t understand the () part.","commit_id":"7460ae30bf92b5bdf68137bc5de9f7891a33b89b"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"c40706d2d345564d77f77053bcb830cfe2d0bd02","unresolved":false,"context_lines":[{"line_number":66,"context_line":"        # LegacyEngineFacade) asap since it\u0027s not compatible with SQLAlchemy"},{"line_number":67,"context_line":"        # 2.0"},{"line_number":68,"context_line":"        conf[\u0027autocommit\u0027] \u003d True"},{"line_number":69,"context_line":"        _FACADES[cache_name] \u003d session.EngineFacade("},{"line_number":70,"context_line":"            connection,"},{"line_number":71,"context_line":"            **conf"},{"line_number":72,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"d9118283_8ec2e5e7","line":69,"range":{"start_line":69,"start_character":39,"end_line":69,"end_character":51},"in_reply_to":"a2c942dc_0af9773c","updated":"2022-09-05 19:07:22.000000000","message":"It seems like it is an alias on the oslo.db side for backwards compatibility.\n\u003e EngineFacade \u003d enginefacade.LegacyEngineFacade\nhttps://github.com/openstack/oslo.db/blob/9119edc86ffa207bdf244e0d21e163c96a06ee8e/oslo_db/sqlalchemy/session.py#L189","commit_id":"7460ae30bf92b5bdf68137bc5de9f7891a33b89b"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"783e731d19901e2b7fff46effb72d7d8f86d4c03","unresolved":false,"context_lines":[{"line_number":66,"context_line":"        # LegacyEngineFacade) asap since it\u0027s not compatible with SQLAlchemy"},{"line_number":67,"context_line":"        # 2.0"},{"line_number":68,"context_line":"        conf[\u0027autocommit\u0027] \u003d True"},{"line_number":69,"context_line":"        _FACADES[cache_name] \u003d session.EngineFacade("},{"line_number":70,"context_line":"            connection,"},{"line_number":71,"context_line":"            **conf"},{"line_number":72,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"63501037_f3d7f802","line":69,"range":{"start_line":69,"start_character":39,"end_line":69,"end_character":51},"in_reply_to":"d9118283_8ec2e5e7","updated":"2022-09-06 07:14:56.000000000","message":"Yeah, it\u0027s just an alias.","commit_id":"7460ae30bf92b5bdf68137bc5de9f7891a33b89b"}]}