)]}'
{"cinder/objects/volume.py":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"b1b5356888a391fa9c1dd47b08f90582d97f0c10","unresolved":true,"context_lines":[{"line_number":516,"context_line":"        # in the DB but now pointing to the migrated volume."},{"line_number":517,"context_line":"        skip \u003d ({\u0027id\u0027, \u0027provider_location\u0027, \u0027glance_metadata\u0027, \u0027use_quota\u0027,"},{"line_number":518,"context_line":"                 \u0027volume_type\u0027, \u0027volume_attachment\u0027}"},{"line_number":519,"context_line":"                | set(self.obj_extra_fields))"},{"line_number":520,"context_line":"        for key in set(dest_volume.fields.keys()) - skip:"},{"line_number":521,"context_line":"            # Only swap attributes that are already set.  We do not want to"},{"line_number":522,"context_line":"            # unexpectedly trigger a lazy-load."}],"source_content_type":"text/x-python","patch_set":8,"id":"a778e949_62541b72","line":519,"range":{"start_line":519,"start_character":16,"end_line":519,"end_character":18},"updated":"2021-09-22 18:52:01.000000000","message":"It\u0027s the first time I see bitwise operators in cinder code.","commit_id":"e07bf3787a7c1a8a55b8f69290cf6166719f6a5e"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cc94f16db7aa67c666200e431447e2a6ee0ed8c4","unresolved":true,"context_lines":[{"line_number":516,"context_line":"        # in the DB but now pointing to the migrated volume."},{"line_number":517,"context_line":"        skip \u003d ({\u0027id\u0027, \u0027provider_location\u0027, \u0027glance_metadata\u0027, \u0027use_quota\u0027,"},{"line_number":518,"context_line":"                 \u0027volume_type\u0027, \u0027volume_attachment\u0027}"},{"line_number":519,"context_line":"                | set(self.obj_extra_fields))"},{"line_number":520,"context_line":"        for key in set(dest_volume.fields.keys()) - skip:"},{"line_number":521,"context_line":"            # Only swap attributes that are already set.  We do not want to"},{"line_number":522,"context_line":"            # unexpectedly trigger a lazy-load."}],"source_content_type":"text/x-python","patch_set":8,"id":"327365ee_f99f7222","line":519,"range":{"start_line":519,"start_character":16,"end_line":519,"end_character":18},"in_reply_to":"a778e949_62541b72","updated":"2021-09-29 18:06:14.000000000","message":"Since Sofia pointed this out, I wonder if there\u0027s any point in using the union function here instead of the operator.  It takes an iterable, and you wouldn\u0027t have to convert obj_extra_fields to a set before doing the union.","commit_id":"e07bf3787a7c1a8a55b8f69290cf6166719f6a5e"}]}