なるようになるブログ

読書感想文かrailsについてかrubyについてか

rails commit log流し読み(2016/11/06)

2016/11/06分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

activerecord/CHANGELOG.md


Merge pull request #26969 from y-yagi/remove_unused_requires

railties/lib/rails/commands/dbconsole/dbconsole_command.rbの修正です。

使用していないrequireを削除しています。


Avoid unscope(:order) when limit_value is presented for count

activerecord/lib/active_record/relation/calculations.rbの修正です。

calculations処理を行う際必ずunscope(:order)を呼び出していたのを、limit_valueが指定されている場合はunscope(:order)を行わないよう修正しています。

COUNTする際のORDER BYはパフォーマンスに影響がある為、unscopeされてしまうと問題になるケースがある為との事です。


Erubis is not actually used in AR

activerecord/test/support/config.rbの修正です。

config fileの読み込み処理でErubisを使用していたのをerbを使用するよう修正しています。Active Recordのlibではerb`を使用しているので、合わせる為。


Merge pull request #26909 from matthewd/query-cache-connection

Active Recordの修正です。

ActiveRecord::ConnectionAdapters::ConnectionPool#checkin(database connectionをpoolに戻すメソッド)の後にquery cacheの無効化処理を行うよう修正しています。

元々はActiveRecord::QueryCache.completeでquery cacheの無効化処理を行っていたのですが、checkin処理の後にやる方が適切だろう、という事で修正されたようです。