)]}'
{"dragonflow/cli/df_db.py":[{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"b14c8054905ecd88bc600ba7e8fa088d995c32f4","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    try:"},{"line_number":50,"context_line":"        keys \u003d db_driver.get_all_keys(table)"},{"line_number":51,"context_line":"    except df_exceptions.DBKeyNotFound:"},{"line_number":52,"context_line":"        keys \u003d []"},{"line_number":53,"context_line":"    print (\u0027 \u0027)"},{"line_number":54,"context_line":"    print (\u0027Keys for table \u0027 + table)"},{"line_number":55,"context_line":"    print (\u0027------------------------------------------------------------\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa0719c6_9e5b0ff6","line":52,"updated":"2016-03-23 11:49:17.000000000","message":"Consider adding a warning that the table was not found anyway. It will help finding typos. While also handled by the caller, it may still prove helpful for debugging.","commit_id":"7d935802bafb3c8de7885eeb19404ba8585bf17a"},{"author":{"_account_id":11343,"name":"Gal Sagie","email":"gal.sagie@gmail.com","username":"GalSagie"},"change_message_id":"28ccf48171b91fd4e5458012d13e83f5383bba24","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    try:"},{"line_number":50,"context_line":"        keys \u003d db_driver.get_all_keys(table)"},{"line_number":51,"context_line":"    except df_exceptions.DBKeyNotFound:"},{"line_number":52,"context_line":"        keys \u003d []"},{"line_number":53,"context_line":"    print (\u0027 \u0027)"},{"line_number":54,"context_line":"    print (\u0027Keys for table \u0027 + table)"},{"line_number":55,"context_line":"    print (\u0027------------------------------------------------------------\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa0719c6_e7c6fa98","line":52,"in_reply_to":"fa0719c6_9e5b0ff6","updated":"2016-03-23 12:30:14.000000000","message":"Table not found and empty table are just the same, tables should be logical entities, the CLI tool verify that the parameter is a name of a valid DB table before calling this method","commit_id":"7d935802bafb3c8de7885eeb19404ba8585bf17a"}]}
