)]}'
{"neutron/plugins/ml2/plugin.py":[{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"83dd633dede53e1ad0aed559e849fbc3ff43ba39","unresolved":false,"context_lines":[{"line_number":644,"context_line":"                                                net_data[az_ext.AZ_HINTS])"},{"line_number":645,"context_line":"                res \u003d super(Ml2Plugin, self).update_network(context,"},{"line_number":646,"context_line":"                    result[\u0027id\u0027], {\u0027network\u0027: {az_ext.AZ_HINTS: az_hints}})"},{"line_number":647,"context_line":"                result[az_ext.AZ_HINTS] \u003d res[az_ext.AZ_HINTS]"},{"line_number":648,"context_line":""},{"line_number":649,"context_line":"            # Update the transparent vlan if configured"},{"line_number":650,"context_line":"            if utils.is_extension_supported(self, \u0027vlan-transparent\u0027):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_fa080482","line":647,"updated":"2015-12-13 06:32:58.000000000","message":"What if the resulted dict doesn\u0027t has \u0027az_ext.AZ_HINTS\u0027 key? Shouldn\u0027t we do getattr to avoid KeyError issues?","commit_id":"999ee86deaab2d72069f9d16c3a7d893c1426fc4"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"2c0845904b15de46d682e72d645cc7a56866b9f1","unresolved":false,"context_lines":[{"line_number":644,"context_line":"                                                net_data[az_ext.AZ_HINTS])"},{"line_number":645,"context_line":"                res \u003d super(Ml2Plugin, self).update_network(context,"},{"line_number":646,"context_line":"                    result[\u0027id\u0027], {\u0027network\u0027: {az_ext.AZ_HINTS: az_hints}})"},{"line_number":647,"context_line":"                result[az_ext.AZ_HINTS] \u003d res[az_ext.AZ_HINTS]"},{"line_number":648,"context_line":""},{"line_number":649,"context_line":"            # Update the transparent vlan if configured"},{"line_number":650,"context_line":"            if utils.is_extension_supported(self, \u0027vlan-transparent\u0027):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_aad21f5a","line":647,"in_reply_to":"7a740942_fa080482","updated":"2015-12-14 01:39:57.000000000","message":"I cannot find the case it happens. If it should happen, we would have to prepare property exception in different layer.","commit_id":"999ee86deaab2d72069f9d16c3a7d893c1426fc4"}]}