)]}'
{"oslo_versionedobjects/checks.py":[{"author":{"_account_id":1297,"name":"Joshua Harlow","email":"jxharlow@godaddy.com","username":"harlowja"},"change_message_id":"61f96a360f947864b67298e0306fcccc036e6998","unresolved":false,"context_lines":[{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"class ObjectVersionChecker(object):"},{"line_number":94,"context_line":"    def _find_remotable_method(self, cls, thing, parent_was_remotable\u003dFalse):"},{"line_number":95,"context_line":"        \"\"\"Follow a chain of remotable things down to the original function.\"\"\""},{"line_number":96,"context_line":"        if isinstance(thing, classmethod):"},{"line_number":97,"context_line":"            return self._find_remotable_method(cls, thing.__get__(None, cls))"}],"source_content_type":"text/x-python","patch_set":6,"id":"da86d52c_ed958fe4","line":94,"updated":"2015-02-18 04:07:59.000000000","message":"Just a curosity question.\n\nRemotable methods are decorated with @remotable (I think that\u0027s right but correct me if I am wrong).\n\nOnce @six.wraps is used u can get a the original function/method that was decorated without going through this. Seems like that should be useful to avoid this jumping right?\n\nSee https://docs.python.org/3/library/functools.html#functools.update_wrapper (search for __wrapped__); I am wondering if that can just be used instead?","commit_id":"7797e5e6149e23fa2273422d3498e7d4fdb8e8b4"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"1e2977ce32ed787ebfb6ed77bd2e6a4a40a77808","unresolved":false,"context_lines":[{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"class ObjectVersionChecker(object):"},{"line_number":94,"context_line":"    def _find_remotable_method(self, cls, thing, parent_was_remotable\u003dFalse):"},{"line_number":95,"context_line":"        \"\"\"Follow a chain of remotable things down to the original function.\"\"\""},{"line_number":96,"context_line":"        if isinstance(thing, classmethod):"},{"line_number":97,"context_line":"            return self._find_remotable_method(cls, thing.__get__(None, cls))"}],"source_content_type":"text/x-python","patch_set":6,"id":"da86d52c_7e0a1421","line":94,"in_reply_to":"da86d52c_ed958fe4","updated":"2015-02-18 15:02:26.000000000","message":"Probably. This is code that is just being moved wholesale from the tests/ module. So if it\u0027s cool, let\u0027s just move it here as part of this patch to generalize the test, and then we can go back and clean that up. I think I do this monkeywork somewhere else, so I can look for that and probably do the same cleanup.","commit_id":"7797e5e6149e23fa2273422d3498e7d4fdb8e8b4"}]}