Stylistically, would it be better to have execute_sql(None) or execute_sql(NO_RESULTS)? I feel that NO_RESULTS makes the code self explanatory without needing to remember that None is the result_type.