)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"2386f1d13a430e589e7463a0a1edf1d8a62d0d0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4b565dae_9abf661e","updated":"2023-03-01 12:43:45.000000000","message":"I am not a data base expert, but it LGTM.","commit_id":"df90b6fcd7a923741099823222845800c81fcf8c"}],"octavia/db/migration/alembic_migrations/versions/9bf4d21caaea_adding_amphora_id_to_listener_.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"64320730f31061669a8b70227f0a54b5d6fa6647","unresolved":true,"context_lines":[{"line_number":42,"context_line":"    bind \u003d op.get_bind()"},{"line_number":43,"context_line":"    inspector \u003d reflection.Inspector.from_engine(bind)"},{"line_number":44,"context_line":"    pk_constraint \u003d inspector.get_pk_constraint(\u0027listener_statistics\u0027)"},{"line_number":45,"context_line":"    op.drop_constraint(pk_constraint.get(\u0027name\u0027), \u0027listener_statistics\u0027,"},{"line_number":46,"context_line":"                       type_\u003d\u0027primary\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    op.create_primary_key(\u0027pk_listener_statistics\u0027, \u0027listener_statistics\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"9026e5c1_ef2c03cb","line":45,"updated":"2023-09-28 15:45:24.000000000","message":"Why are we completely dropping this constraint?","commit_id":"df90b6fcd7a923741099823222845800c81fcf8c"}],"octavia/db/migration/alembic_migrations/versions/e37941b010db_add_lb_flavor_constraint.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"64320730f31061669a8b70227f0a54b5d6fa6647","unresolved":true,"context_lines":[{"line_number":70,"context_line":"    # Note: constant is not used here to not trigger security tool errors."},{"line_number":71,"context_line":"    bind \u003d op.get_bind()"},{"line_number":72,"context_line":"    if bind.dialect.name \u003d\u003d \"postgresql\":"},{"line_number":73,"context_line":"        op.execute(\"UPDATE load_balancer \""},{"line_number":74,"context_line":"                   \"SET flavor_id \u003d \u002700000000-0000-0000-0000-000000000000\u0027 \""},{"line_number":75,"context_line":"                   \"FROM flavor \""},{"line_number":76,"context_line":"                   \"WHERE flavor.id IS NULL \""}],"source_content_type":"text/x-python","patch_set":3,"id":"9e0096d0_67f59ab6","line":73,"updated":"2023-09-28 15:45:24.000000000","message":"This is not good SQL syntax. it should call out the join.","commit_id":"df90b6fcd7a923741099823222845800c81fcf8c"}]}
