Don't explain except normal CRUD sql. #7657

Merged
merged 1 commit into from Sep 16, 2012

Projects

None yet

4 participants

@kennyj
Contributor
kennyj commented Sep 16, 2012

In current implementation, it's break when executing un-explainable query.
These query should be ignored for explaining.

closes #7544 #6458

/cc @rafaelfranca

@rafaelfranca
Member

Could you add Changelog entry?

@fxn
Member
fxn commented Sep 16, 2012

Could the anchor be \A instead of ^ please?

@kennyj
Contributor
kennyj commented Sep 16, 2012

@rafaelfranca @fxn I updated this.

@drogus drogus merged commit 82efe89 into rails:master Sep 16, 2012
@rafaelfranca rafaelfranca added a commit that referenced this pull request Sep 17, 2012
@rafaelfranca rafaelfranca Backport explain fixes.
* Mark as SCHEMA some schema database queries. #7648
* Don't explain queries except normal CRUD sql. #7657

Closes #6458
Closes #7544
6abe0e7
@rafaelfranca rafaelfranca added a commit that referenced this pull request Sep 17, 2012
@rafaelfranca @sikachu rafaelfranca + sikachu Backport explain fixes.
* Mark as SCHEMA some schema database queries. #7648
* Don't explain queries except normal CRUD sql. #7657

Closes #6458
Closes #7544
b9329a2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment