)]}'
{"ceilometer/storage/impl_sqlalchemy.py":[{"author":{"_account_id":7478,"name":"Nadya Shakhat","email":"nadmi4@gmail.com","username":"nprivalova"},"change_message_id":"6baf6eb4cfb4ac207670bb93245a3c9747070885","unresolved":false,"context_lines":[{"line_number":155,"context_line":"        if sample_filter.end_timestamp_op \u003d\u003d \u0027le\u0027:"},{"line_number":156,"context_line":"            query \u003d query.filter(models.Sample.timestamp \u003c\u003d ts_end)"},{"line_number":157,"context_line":"        else:"},{"line_number":158,"context_line":"            query \u003d query.filter(models.Sample.timestamp \u003c ts_end)"},{"line_number":159,"context_line":"    if sample_filter.user is not None:"},{"line_number":160,"context_line":"        if sample_filter.user \u003d\u003d \u0027None\u0027:"},{"line_number":161,"context_line":"            sample_filter.user \u003d None"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a5de9d1_49a2659c","line":158,"updated":"2016-02-02 09:33:22.000000000","message":"Isn\u0027t it enough to use two ifs:\n\nif sample_filter.user \u003d\u003d \u0027None\u0027:\n          sample_filter.user \u003d None\n\nbecause if sample_filter.user \u003d\u003d \u0027None\u0027 it automatically means that sample_filter.user is not None","commit_id":"f2dd036402a5f47562eb36881a7771c0897c2101"}]}