@byroot @eileencodes Definitely true. I was just wondering if you had any feelings about `#exec_query, #internal_exec_query, etc` because you had refactored those a bit.

@byroot I was hoping to get your thoughts on something, if you don't mind. At Zendesk we're thinking of opening a PR to refactor some ActiveRecord code you've been working on recently.

What we'd like to do:
- Perhaps move `#check_if_write_query` to be called earlier in the execution stack. Right now that method gets called in multiple places. It could _potentially_ simplify things if it was abstracted somewhere above (#internal_exec_query, #exec_insert, etc).