2020/03/06分のコミットです。
CHANGELOGへの追加はありませんでした。
parent 6d0895a4894724e1a923a514daad8fb3c9ac2c28
activejob/lib/active_job/exceptions.rb
の修正です。
retry_on
のjitter
オプションに0以外の値を指定してもrandomな値にならない場合があったの(Kernel.rand
に渡す値が1-2の値になってしまって結果jitterの値が0になっていた)を修正しています。
Merge pull request #37808 from headius/optimize_string_inquirer
Active Support、railtiesの修正です。
Rails.env
、Rails.env=
で値を取得/指定する際にActiveSupport::StringInquirer
を使用していたのを、envを取得する為のActiveSupport::EnvironmentInquirer
クラスを追加し、そちらを使用するよう修正しています。
ActiveSupport::StringInquirer
はmethod_missing
を使用している、かつ、method_missing
は遅い処理な為、Rails.env.devlopment?
のようなチェック処理も比較的遅い処理になってしまっていました。
ActiveSupport::EnvironmentInquirer
ではデフォルトのenv("development", "test", "production")対してActiveSupport::StringInquirer
で実行されるのと同等のメソッドを先に定義しておいて、method_missing
が実行されるのを避けるようにしています。デフォルト以外のenvに対する処理は変わらず。
Revert "Merge pull request #37808 from headius/optimize_string_inquirer"
先のActiveSupport::EnvironmentInquirer
を追加した対応をRevertしています。CIが不安定になってしまった為。後ほど再度コミットされています。
Refactor for_each(databases) code
activerecord/lib/active_record/railties/databases.rake
、
activerecord/lib/active_record/tasks/database_tasks.rb
の修正です。
for_each
の戻り値をname
からdb_config
に変更し、db_config
が必要な場合に再度name
から取得しないで済むよう修正しています。
が、name
だけ返していたのは意図的らしく、後ほどRevertされています。
Use optimized subclass of StringInquirer for Rails.env
Active Support、railtiesの修正です。
ActiveSupport::EnvironmentInquirer
について、実装をちょっと修正&autoloadの指定に追加し再度コミットしています。
rails guideのDebugging Rails Applications
の修正です。
Add logging performance [ci skip]で追加されたログの性能に関する説明がStackOverflowのanswer(https://stackoverflow.com/questions/16546730/logging-in-rails-is-there-any-performance-hit/16546935#16546935)からの盗用だったという事で、元のanswerを書かれた人の名前と StackOverflowへのリンクを引用元として記載するよう修正しています。
Revert "Merge pull request #38658 from eileencodes/refactor-for-each-databases-code"
for_each
の戻り値をname
からdb_config
に変更した対応をRevertしています。
Merge pull request #38665 from roramirez/test-master-new-app
railties/test/generators/app_generator_test.rb
の修正です。
rails new
に--master
オプションを指定した場合のテストを追加しています。
Fix database configruation for multi-db
railties/test/isolation/abstract_unit.rb
の修正です。
multi-db用のconfigの指定で、replicaの指定に誤りがあった(replica
がreadonly
になっていた)のを修正しています。