)]}'
{"gnocchi/indexer/sqlalchemy.py":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"a1985889f97d756fe7541b28482fd221568a3948","unresolved":false,"context_lines":[{"line_number":337,"context_line":"                    resource_cols[col.name] \u003d sqlalchemy.bindparam("},{"line_number":338,"context_line":"                        col.name, value, col.type).label(col.name)"},{"line_number":339,"context_line":"                else:"},{"line_number":340,"context_line":"                    history_cols[col.name] \u003d col"},{"line_number":341,"context_line":"                    resource_cols[col.name] \u003d getattr(resource_cls,"},{"line_number":342,"context_line":"                                                      col.name)"},{"line_number":343,"context_line":"            s1 \u003d sqlalchemy.select(history_cols.values())"}],"source_content_type":"text/x-python","patch_set":3,"id":"9aa53dc9_b8c5645c","line":340,"updated":"2015-04-07 11:39:38.000000000","message":"Looks like it\u0027s common","commit_id":"ef7f70ec711f1026f47f7e6e68834dc99e4dc744"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"6e9848162278201ef6f564f3845d3e95779ef6e5","unresolved":false,"context_lines":[{"line_number":337,"context_line":"                    resource_cols[col.name] \u003d sqlalchemy.bindparam("},{"line_number":338,"context_line":"                        col.name, value, col.type).label(col.name)"},{"line_number":339,"context_line":"                else:"},{"line_number":340,"context_line":"                    history_cols[col.name] \u003d col"},{"line_number":341,"context_line":"                    resource_cols[col.name] \u003d getattr(resource_cls,"},{"line_number":342,"context_line":"                                                      col.name)"},{"line_number":343,"context_line":"            s1 \u003d sqlalchemy.select(history_cols.values())"}],"source_content_type":"text/x-python","patch_set":3,"id":"9aa53dc9_16c65c67","line":340,"in_reply_to":"9aa53dc9_b8c5645c","updated":"2015-04-07 14:05:03.000000000","message":"Done","commit_id":"ef7f70ec711f1026f47f7e6e68834dc99e4dc744"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"a1985889f97d756fe7541b28482fd221568a3948","unresolved":false,"context_lines":[{"line_number":370,"context_line":""},{"line_number":371,"context_line":"        session \u003d self.engine_facade.get_session()"},{"line_number":372,"context_line":""},{"line_number":373,"context_line":"        if not history:"},{"line_number":374,"context_line":"            target_cls \u003d self._resource_type_to_class(resource_type)"},{"line_number":375,"context_line":"            q \u003d session.query(target_cls)"},{"line_number":376,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9aa53dc9_982ce04b","line":373,"updated":"2015-04-07 11:39:38.000000000","message":"I tend to prefer if history: else as it\u0027s shorter :)","commit_id":"ef7f70ec711f1026f47f7e6e68834dc99e4dc744"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"6e9848162278201ef6f564f3845d3e95779ef6e5","unresolved":false,"context_lines":[{"line_number":370,"context_line":""},{"line_number":371,"context_line":"        session \u003d self.engine_facade.get_session()"},{"line_number":372,"context_line":""},{"line_number":373,"context_line":"        if not history:"},{"line_number":374,"context_line":"            target_cls \u003d self._resource_type_to_class(resource_type)"},{"line_number":375,"context_line":"            q \u003d session.query(target_cls)"},{"line_number":376,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9aa53dc9_d6a6b4c0","line":373,"in_reply_to":"9aa53dc9_982ce04b","updated":"2015-04-07 14:05:03.000000000","message":"Done","commit_id":"ef7f70ec711f1026f47f7e6e68834dc99e4dc744"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"a1985889f97d756fe7541b28482fd221568a3948","unresolved":false,"context_lines":[{"line_number":375,"context_line":"            q \u003d session.query(target_cls)"},{"line_number":376,"context_line":"        else:"},{"line_number":377,"context_line":"            target_cls \u003d self._get_history_result_mapper(resource_type)"},{"line_number":378,"context_line":"            q \u003d session.query(target_cls)"},{"line_number":379,"context_line":""},{"line_number":380,"context_line":"        if attribute_filter:"},{"line_number":381,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9aa53dc9_382b9432","line":378,"updated":"2015-04-07 11:39:38.000000000","message":"This is common with the other block","commit_id":"ef7f70ec711f1026f47f7e6e68834dc99e4dc744"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"6e9848162278201ef6f564f3845d3e95779ef6e5","unresolved":false,"context_lines":[{"line_number":375,"context_line":"            q \u003d session.query(target_cls)"},{"line_number":376,"context_line":"        else:"},{"line_number":377,"context_line":"            target_cls \u003d self._get_history_result_mapper(resource_type)"},{"line_number":378,"context_line":"            q \u003d session.query(target_cls)"},{"line_number":379,"context_line":""},{"line_number":380,"context_line":"        if attribute_filter:"},{"line_number":381,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9aa53dc9_b6ab3089","line":378,"in_reply_to":"9aa53dc9_382b9432","updated":"2015-04-07 14:05:03.000000000","message":"Done","commit_id":"ef7f70ec711f1026f47f7e6e68834dc99e4dc744"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"7d822b19a5e9ce56ab0b7ad074f374f77d6d5324","unresolved":false,"context_lines":[{"line_number":324,"context_line":"            return self._resource_to_dict(r, with_metrics)"},{"line_number":325,"context_line":""},{"line_number":326,"context_line":"    def _get_history_result_mapper(self, resource_type):"},{"line_number":327,"context_line":"            resource_cls \u003d self._resource_type_to_class(resource_type)"},{"line_number":328,"context_line":"            history_cls \u003d self._resource_type_to_class(resource_type,"},{"line_number":329,"context_line":"                                                       \u0027history\u0027)"},{"line_number":330,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9aa53dc9_bcc1cfe6","line":327,"updated":"2015-04-07 14:42:38.000000000","message":"This entire block is indented one too many isn\u0027t it?","commit_id":"1319fa0831450d1bcc59b493b6c58ab5616aa98f"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"71992cd3d5205a334eac48b5d7a26a45a4a80168","unresolved":false,"context_lines":[{"line_number":295,"context_line":"            q \u003d q.options(sqlalchemy.orm.joinedload(resource_cls.metrics))"},{"line_number":296,"context_line":"        return q.first()"},{"line_number":297,"context_line":""},{"line_number":298,"context_line":"    def _get_history_result_mapper(self, resource_type):"},{"line_number":299,"context_line":"            resource_cls \u003d self._resource_type_to_class(resource_type)"},{"line_number":300,"context_line":"            history_cls \u003d self._resource_type_to_class(resource_type,"},{"line_number":301,"context_line":"                                                       \u0027history\u0027)"}],"source_content_type":"text/x-python","patch_set":19,"id":"9aa53dc9_af512a38","line":298,"updated":"2015-04-09 17:14:41.000000000","message":"Is this extra indentation intentional?","commit_id":"0dfb9b542259f5d305a0f20c14b721392a0e532b"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"a16c7b004a6052bd92d394356d8f15486785a963","unresolved":false,"context_lines":[{"line_number":295,"context_line":"            q \u003d q.options(sqlalchemy.orm.joinedload(resource_cls.metrics))"},{"line_number":296,"context_line":"        return q.first()"},{"line_number":297,"context_line":""},{"line_number":298,"context_line":"    def _get_history_result_mapper(self, resource_type):"},{"line_number":299,"context_line":"            resource_cls \u003d self._resource_type_to_class(resource_type)"},{"line_number":300,"context_line":"            history_cls \u003d self._resource_type_to_class(resource_type,"},{"line_number":301,"context_line":"                                                       \u0027history\u0027)"}],"source_content_type":"text/x-python","patch_set":19,"id":"9aa53dc9_52bc6cdb","line":298,"in_reply_to":"9aa53dc9_af512a38","updated":"2015-04-10 09:09:12.000000000","message":"Done","commit_id":"0dfb9b542259f5d305a0f20c14b721392a0e532b"}]}