Project

General

Profile

« Previous | Next » 

Revision 10195

lib/sql_io.py: cleanup_table(): added assertion that the table exists, so that if it doesn't, the error will occur as part of an assertion rather than as part of the util.table_nulls_mapped__get() call, which might confusingly lead users to believe that this is a bug in util.table_nulls_mapped__get() when in fact the problem is that the table is not installed

View differences:

lib/sql_io.py
37 37
def cleanup_table(db, table):
38 38
    '''idempotent'''
39 39
    table = sql_gen.as_Table(table)
40
    assert sql.table_exists(db, table)
40 41
    
41 42
    if table_nulls_mapped__get(db, table): return # already cleaned up
42 43
    

Also available in: Unified diff