)]}'
{"tacker/api/validation/__init__.py":[{"author":{"_account_id":18955,"name":"dharmendra","email":"dharmendra.kushwaha@gmail.com","username":"dharmendra"},"change_message_id":"3c0dd946c0afda82ae8183733e2d80a385693b89","unresolved":false,"context_lines":[{"line_number":42,"context_line":"            try:"},{"line_number":43,"context_line":"                schema_validator.validate(kwargs[\u0027body\u0027])"},{"line_number":44,"context_line":"            except KeyError:"},{"line_number":45,"context_line":"                return webob.exc.HTTPBadRequest("},{"line_number":46,"context_line":"                    explanation\u003d_(\"Malformed request body\"))"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"            return func(*args, **kwargs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_2f090530","line":45,"range":{"start_line":45,"start_character":16,"end_line":45,"end_character":22},"updated":"2019-09-02 17:53:04.000000000","message":"raise ?","commit_id":"307fe63a5a6082b2ecf4b5388532ed978f220139"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"2322340455652cac6e8dd1c4a9315659ab297c75","unresolved":false,"context_lines":[{"line_number":42,"context_line":"            try:"},{"line_number":43,"context_line":"                schema_validator.validate(kwargs[\u0027body\u0027])"},{"line_number":44,"context_line":"            except KeyError:"},{"line_number":45,"context_line":"                return webob.exc.HTTPBadRequest("},{"line_number":46,"context_line":"                    explanation\u003d_(\"Malformed request body\"))"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"            return func(*args, **kwargs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_f4bc0bb6","line":45,"range":{"start_line":45,"start_character":16,"end_line":45,"end_character":22},"in_reply_to":"7faddb67_2f090530","updated":"2019-09-04 04:03:36.000000000","message":"Will fix this comment in the next PS.","commit_id":"307fe63a5a6082b2ecf4b5388532ed978f220139"}],"tacker/api/validation/validators.py":[{"author":{"_account_id":18955,"name":"dharmendra","email":"dharmendra.kushwaha@gmail.com","username":"dharmendra"},"change_message_id":"3c0dd946c0afda82ae8183733e2d80a385693b89","unresolved":false,"context_lines":[{"line_number":73,"context_line":"            verb \u003d \"was\""},{"line_number":74,"context_line":"        else:"},{"line_number":75,"context_line":"            verb \u003d \"were\""},{"line_number":76,"context_line":"        yield jsonschema_exc.ValidationError("},{"line_number":77,"context_line":"            error % (\", \".join(repr(extra) for extra in extra_properties),"},{"line_number":78,"context_line":"                     verb))"},{"line_number":79,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_dd1273fd","line":76,"range":{"start_line":76,"start_character":7,"end_line":76,"end_character":45},"updated":"2019-09-02 17:53:04.000000000","message":"why yield instead of raise?","commit_id":"307fe63a5a6082b2ecf4b5388532ed978f220139"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"2322340455652cac6e8dd1c4a9315659ab297c75","unresolved":false,"context_lines":[{"line_number":73,"context_line":"            verb \u003d \"was\""},{"line_number":74,"context_line":"        else:"},{"line_number":75,"context_line":"            verb \u003d \"were\""},{"line_number":76,"context_line":"        yield jsonschema_exc.ValidationError("},{"line_number":77,"context_line":"            error % (\", \".join(repr(extra) for extra in extra_properties),"},{"line_number":78,"context_line":"                     verb))"},{"line_number":79,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_74b1fbdf","line":76,"range":{"start_line":76,"start_character":7,"end_line":76,"end_character":45},"in_reply_to":"7faddb67_dd1273fd","updated":"2019-09-04 04:03:36.000000000","message":"raise will work too. Infact, we don\u0027t need this method at this point of time. So I will delete this method in the next PS.","commit_id":"307fe63a5a6082b2ecf4b5388532ed978f220139"},{"author":{"_account_id":18955,"name":"dharmendra","email":"dharmendra.kushwaha@gmail.com","username":"dharmendra"},"change_message_id":"3c0dd946c0afda82ae8183733e2d80a385693b89","unresolved":false,"context_lines":[{"line_number":132,"context_line":"    passed through cinder api commonly."},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"    \"\"\""},{"line_number":135,"context_line":"    validator \u003d None"},{"line_number":136,"context_line":"    validator_org \u003d jsonschema.Draft4Validator"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"    def __init__(self, schema, relax_additional_properties\u003dFalse):"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_00541e53","line":135,"range":{"start_line":135,"start_character":3,"end_line":135,"end_character":20},"updated":"2019-09-02 17:53:04.000000000","message":"redundant. As it already getting initialized in __init__ , can be removed from here.","commit_id":"307fe63a5a6082b2ecf4b5388532ed978f220139"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"2322340455652cac6e8dd1c4a9315659ab297c75","unresolved":false,"context_lines":[{"line_number":132,"context_line":"    passed through cinder api commonly."},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"    \"\"\""},{"line_number":135,"context_line":"    validator \u003d None"},{"line_number":136,"context_line":"    validator_org \u003d jsonschema.Draft4Validator"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"    def __init__(self, schema, relax_additional_properties\u003dFalse):"}],"source_content_type":"text/x-python","patch_set":2,"id":"7faddb67_34c70348","line":135,"range":{"start_line":135,"start_character":3,"end_line":135,"end_character":20},"in_reply_to":"7faddb67_00541e53","updated":"2019-09-04 04:03:36.000000000","message":"Will fix this comment in the next PS.","commit_id":"307fe63a5a6082b2ecf4b5388532ed978f220139"}]}
