)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"767825b27cca42064129ad38460d29f5d378c654","unresolved":false,"context_lines":[{"line_number":7,"context_line":"l3_db: not use l2 plugin _get_port unnecessarily"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"l3_db uses l2 plugin _get_port method unnecessarily."},{"line_number":10,"context_line":"It\u0027s dangerous because it may update port db without notifying l2 plugin."},{"line_number":11,"context_line":"In that case, l2 plugin or ML2 mechanism driver will be confused."},{"line_number":12,"context_line":"This patch replace _get_port with get_port method where possible."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a41bdd9_ef478d49","line":10,"updated":"2015-07-17 05:35:34.000000000","message":"why would the get_port() method trigger this?","commit_id":"51aa8e12deb5d36dca7a8c1cf06edcc267498b89"},{"author":{"_account_id":333,"name":"Isaku Yamahata","email":"isaku.yamahata@gmail.com","username":"yamahata"},"change_message_id":"ac889caa4c8fbf05400cad3cf2929e7a2e2c1135","unresolved":false,"context_lines":[{"line_number":7,"context_line":"l3_db: not use l2 plugin _get_port unnecessarily"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"l3_db uses l2 plugin _get_port method unnecessarily."},{"line_number":10,"context_line":"It\u0027s dangerous because it may update port db without notifying l2 plugin."},{"line_number":11,"context_line":"In that case, l2 plugin or ML2 mechanism driver will be confused."},{"line_number":12,"context_line":"This patch replace _get_port with get_port method where possible."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_723b47ec","line":10,"in_reply_to":"9a41bdd9_ef478d49","updated":"2015-07-20 19:53:09.000000000","message":"I suppose you meant _get_port.\nIt returns ORM object with which the caller(l3_db) can update db rows directly.\n\nIf I respined this patch, I\u0027ll update the commit log.","commit_id":"51aa8e12deb5d36dca7a8c1cf06edcc267498b89"}]}