FS-3307 --resolve thanks, in the future, be sure to not include vanity comments.  The intention of the patch will be preserved by the jira ticket.  Also check our code formating guidelines.  Some of the patch was redundant so it was removed, mprintf already asserts when it doesn't work, and execute_sql_now both frees the sql string and locks the mutex for you.