Upgrade to Pro — share decks privately, control downloads, hide ads and more …

Rails Performance Issues and Solutions

Akira Matsuda
September 29, 2019

Rails Performance Issues and Solutions

Slides for my keynote at RubyConf Indonesia 2019 in Jakarta https://ruby.id/conf/2019/ #RubyconfID

Akira Matsuda

September 29, 2019
Tweet

More Decks by Akira Matsuda

Other Decks in Programming

Transcript

  1. The Ruby Team Is Going to Introduce Some Keyword Arguments

    Incompatibilities in Ruby 3 And I'm investigating the new specification on the Rails codebase
  2. # Gemfile @@ -31,8 +31,6 @@ group :development, :test do

    end group :development do - # Access an interactive console on exception pages or by calling 'console' anywhere in the code. - gem 'web-console', '>= 3.3.0' gem 'listen', '>= 3.0.5', '< 3.2' end @@ -43,6 +41,3 @@ group :test do # Easy installation and use of web drivers to run system tests with browsers gem 'webdrivers' end - -# Windows does not include zoneinfo files, so bundle the tzinfo-data gem -gem 'tzinfo-data', platforms: [:mingw, :mswin, :x64_mingw, :jruby]
  3. Starting Up the Rails Server in `production` Mode $ rails

    assets:precompile $ cp db/development.sqlite3 db/production.sqlite3 $ env RAILS_SERVE_STATIC_FILES=1 rails s -e production
  4. Maybe We Can Use time Command? $ time curl http:/

    /localhost:3000/posts/ > /dev/null
  5. Maybe We Can Use Something Like ab or gatling? $

    ab -n 100 -c 10 -l http:/ /127.0.0.1:3000/posts
  6. Which Part of the App Would We Measure? Maybe the

    controllers' `process_action` method?
  7. A Very Rough Monkey-patch for Benchmarking `process_action` # config/initializers/perf.rb ActionController::Base.prepend(

    Module.new { def process_action(*) result = nil p Benchmark.realtime { result = super } result end } )
  8. config/initializers/*.rb I usually draft this sort of quick monkey-patches in

    `config/ initializers/` No need to explicitly `require` No need to give a name Maybe nobody else does this, but works for me...
  9. Now the App Prints Each Request Processing Time But `ab`ing

    with `-n 100` prints 100 lines What we want is a summary of 100 runs
  10. A Benchmarking Script that Iterates the `process_action` # config/initializers/perf.rb ActionController::Base.prepend(

    Module.new { def process_action(*) Benchmark.ips do |x| x.report('process_action') { super } end super end } )
  11. Or We Could Include the Rack Middleware Stack and `before_action`

    and Such # config/initializers/perf.rb Rails::Engine.prepend( Module.new { def call(*) Benchmark.ips do |x| x.report('call') { super } end super end } )
  12. This Means That the Action Runs 240 Times per Second

    Seems like quite an accurate number than the time command one
  13. This Approach Is Quite Flexible We can `Benchmark.ips` any part

    of the application for measuring its speed So we can
 "devide and conquer"
  14. Route to /posts/ok # config/routes.rb Rails.application.routes.draw do - resources :posts

    + resources :posts do + collection do + get :ok + end + end # For details on the DSL available within this file, see https://guides.rubyonrails.org/routing.html end
  15. The Controller Action # app/controllers/posts_controller.rb class PostsController < ApplicationController before_action

    :set_post, only: [:show, :edit, :update, :destroy] + def ok + head :ok + end + # GET /posts # GET /posts.json def index
  16. These Results Tell Us That... Finding a record is slow

    Rendering it is also slow Request handling is slow as well
  17. There Are Some "Numbers" That We Can Get from the

    Ruby Process Time elapsed Memory usage Method invocations System calls etc...
  18. Why Does the Number of Allocations Matters? Allocating Objects in

    Ruby is a very heavy process Plus, it causes GC pressure The number would not be affected by the environment
  19. Actually, Rails 6 Shows This in the Log Completed 200

    OK in 58ms (Views: 9.0ms | ActiveRecord: 2.6ms | Allocations: 7601)
  20. How Can We Measure Object Allocations? $ ruby -e "c

    = GC.stat(:total_allocated_objects); 100.times { _ = 'hello' }; p GC.stat(:total_allocated_objects) - c" 100
  21. Number of Objects Allocated During a Rails Request # config/initializers/perf.rb

    Rails::Engine.prepend( Module.new { def call(*) c = GC.stat(:total_allocated_objects) result = super p GC.stat(:total_allocated_objects) - c result end } )
  22. An Utility Method That Prints the Memory Usage module Prof

    def self.mem result = nil MemoryProfiler.report { result = yield }.pretty_print(retained_strings: 0, allocated_strings: 100, normalize_paths: true) result end end
  23. Result Total allocated: 63830 bytes (599 objects) Total retained: 21391

    bytes (134 objects) allocated memory by gem ----------------------------------- 27104 actionpack-6.0.0/lib 22574 activesupport-6.0.0/lib 5718 rack-2.0.7/lib 1976 actionview-6.0.0/lib 1944 logger 1520 ipaddr 1157 railties-6.0.0/lib 677 securerandom 504 concurrent-ruby-1.1.5/lib 320 activerecord-6.0.0/lib 176 monitor 80 turbolinks-5.2.1 40 i18n-1.6.0/lib 40 uri allocated memory by file ----------------------------------- 7128 actionpack-6.0.0/lib/action_controller/renderer.rb 3702 rack-2.0.7/lib/rack/utils.rb 3624 actionpack-6.0.0/lib/action_dispatch/http/response.rb ...
  24. It Turned Out That, a simple action that just returns

    a "200 OK" header creates 599 objects and consumes 63830 bytes of memory.
  25. Let's See Who Is Instantiating AC::Renderer # actionpack/lib/action_controller/renderer.rb @@ -63,6

    +63,7 @@ module ActionController # It will be merged with the default Rack environment defined by # +ActionController::Renderer::DEFAULTS+. def initialize(controller, env, defaults) + puts caller @controller = controller @defaults = defaults @env = normalize_keys defaults.merge(env)
  26. An Initializer in ActionText::Engine # actiontext-6.0.0/lib/action_text/engine.rb initializer "action_text.renderer" do |app|

    app.executor.to_run { ActionText::Content.renderer = ApplicationController.renderer } app.executor.to_complete { ActionText::Content.renderer = ApplicationController.renderer } ActiveSupport.on_load(:action_text_content) do self.renderer = ApplicationController.renderer end ActiveSupport.on_load(:action_controller_base) do before_action { ActionText::Content.renderer = ApplicationController.renderer.new(request.env) } end end
  27. So,

  28. When Action Text Is Bundled and Required, It Instantiates An

    AC::Renderer Before Each Action In Every Controller
  29. Skip Requiring actiontext/ railtie (unless you use it) # config/application.rb

    require_relative 'boot' -require 'rails/all' +%w( + active_record/railtie + active_storage/engine + action_controller/railtie + action_view/railtie + action_mailer/railtie + active_job/railtie + action_cable/engine + action_mailbox/engine + rails/test_unit/railtie + sprockets/railtie +).each do |railtie| + begin + require railtie + rescue LoadError + end +end
  30. What If We Skip Requiring Some More Other Components That

    We Don't Use Here? # config/application.rb @@ -2,13 +2,10 @@ require_relative 'boot' %w( active_record/railtie - active_storage/engine action_controller/railtie action_view/railtie action_mailer/railtie active_job/railtie - action_cable/engine - action_mailbox/engine
  31. Benchmark Result for Each Action (Allocation) ok: 55560 bytes (574

    objects) show: 127210 bytes (1301 objects) new 173108 bytes (1657 objects) index: 597728 bytes (6333 objects)
  32. Now We Got the Numbers We need such numbers to

    see how we're optimizing things
  33. Now We Have a List of Files that Consume Memory

    allocated memory by file ----------------------------------- 3702 rack-2.0.7/lib/rack/utils.rb 3624 actionpack-6.0.0/lib/action_dispatch/http/response.rb 2944 activesupport-6.0.0/lib/active_support/callbacks.rb 2728 activesupport-6.0.0/lib/active_support/ hash_with_indifferent_access.rb 2720 actionpack-6.0.0/lib/action_dispatch/middleware/cookies.rb 2144 activesupport-6.0.0/lib/active_support/messages/rotator.rb 1944 ruby/lib/logger.rb 1856 activesupport-6.0.0/lib/active_support/parameter_filter.rb 1680 activesupport-6.0.0/lib/active_support/inflector/methods.rb 1520 ruby/lib/ipaddr.rb 1480 actionpack-6.0.0/lib/action_dispatch/journey/router.rb 1280 actionpack-6.0.0/lib/action_controller/metal/ instrumentation.rb
  34. And Now You're Ready to Commit Your Patches $ cd

    (bundle show actionpack) % cd `bundle show actionpack` (zsh)
  35. Regexp#=~ $ ruby -rmemory_profiler -e "s = 'ABC'; MemoryProfiler.report {

    s =~ / A/ }.pretty_print" Total allocated: 208 bytes (2 objects) allocated memory by location ----------------------------------- 208 -e:1 allocated memory by class ----------------------------------- 168 MatchData 40 String allocated objects by class ----------------------------------- 1 MatchData 1 String
  36. Regexp#match? $ ruby -rmemory_profiler -e "s = 'ABC'; MemoryProfiler.report {

    s.match?(/A/) }.pretty_print" Total allocated: 0 bytes (0 objects) Total retained: 0 bytes (0 objects) allocated memory by gem ----------------------------------- NO DATA
  37. Hash#merge $ ruby -rmemory_profiler -e "h1, h2, h3 = {a:

    1}, {a: 2}, {a: 3}; MemoryProfiler.report { h1.merge(h2.merge(h3)) }.pretty_print" Total allocated: 336 bytes (2 objects) allocated memory by class ----------------------------------- 336 Hash allocated objects by class ----------------------------------- 2 Hash
  38. Hash#[]= $ ruby -rmemory_profiler -e "h1, h2 = {a: 1},

    {a: 2}; MemoryProfiler.report { h = h1.merge(h2); h[:a] = 3; h }.pretty_print" Total allocated: 168 bytes (1 objects) allocated memory by class ----------------------------------- 168 Hash allocated objects by class ----------------------------------- 1 Hash
  39. Hash#fetch Taking Two Arguments $ ruby -rmemory_profiler -e "h =

    {x: 1}; MemoryProfiler.report { h.fetch(:x, 'hello') }.pretty_print" Total allocated: 40 bytes (1 objects) allocated memory by class ----------------------------------- 40 String allocated objects by class ----------------------------------- 1 String
  40. Hash#[] || default $ ruby -rmemory_profiler -e "h = {x:

    1}; MemoryProfiler.report { h[:x] || 'hello' }.pretty_print" Total allocated: 0 bytes (0 objects) allocated memory by class ----------------------------------- NO DATA
  41. I Pushed About 60 Micro- Optimization Patches Like These to

    Rails Since the 6.0.0 Release Since I decided to talk about Rails performance in this conference And I think I could reduce
 5ʙ10% memory consumption
  42. So Instead, Let's Bundle Edge Rails and See How Things

    Are Improved in the Current master # Gemfile -gem 'rails', '~> 6.0.0' +gem 'rails', path: 'PATH/TO/RAILS'
  43. The Improvement Was Indeed Only 1% Since 6.0.0 1% of

    the memory usage 3% of the number of the allocated objects
  44. Because It Uses Whole M + V + C Stack

    Where posts/ok uses only "C"
  45. Measuring Active Record Finder Alone # app/controllers/posts_controller.rb private # Use

    callbacks to share common setup or constraints between actions. def set_post - @post = Post.find(params[:id]) + Prof.mem do + @post = Post.find(params[:id]) + end end
  46. Things That Consume Memory in Action View Strings and String-ish

    Objects Template engines Template lookup I18n UrlHelper
  47. ActiveSupport::SafeBuffer A variety of String class that can distinguish "safe"

    string and "unsafe" string Inherits String class to hold a flag as an ivar inside
  48. Can't We Use the taint Flag for This? I tried,

    but it didn't work well Because we cannot create "tainted by default" Strings
  49. ActiveSupport::SafeBuffer We may be able to reduce so much garbage

    in the views if we can improve this Maybe we can create some kind of extension (that reuses the flag bit in String?) in the future
  50. Template Engines Template Engines tend to create so many garbage

    Strings while composing an HTML body String
  51. But Has the Tradeoff It has no auto HTML safety

    guard We need to manually call `h()` just like we used to do before Rails 3
  52. Stringification -<p id="notice"><%= notice %></p> +<p id="notice">#{h notice }</p> <p>

    <strong>Title:</strong> - <%= @post.title %> + #{h @post.title } </p> -<%= link_to 'Edit', edit_post_path(@post) %> | -<%= link_to 'Back', posts_path %> +#{ link_to 'Edit', edit_post_path(@post) } | +#{ link_to 'Back', posts_path }
  53. Memory Usage Decreased, But Allocated Objects Increased...! Because string_template
 could

    create some extra String / SafeBuffer objects when concatenating strings with explicit `h()` call
  54. string_template Maybe efficient in some particular cases Inside heavy loops

    With lots of string interpolations That needs no HTML escaping But Action View is not generally optimized for this template engine
  55. Who Consumes That Much Memory? allocated memory by file -----------------------------------

    8088 i18n-1.6.0/lib/i18n/interpolate/ruby.rb 7272 i18n-1.6.0/lib/i18n/backend/base.rb 6396 actionview/lib/action_view/helpers/tag_helper.rb 5088 actionview/lib/action_view/lookup_context.rb 5060 rack-2.0.7/lib/rack/utils.rb 4944 activesupport/lib/active_support/core_ext/string/output_safety.rb 4552 actionview/lib/action_view/helpers/form_helper.rb
  56. I18n.t(:hello) => "͜Μʹͪ͸" A very simple function that looks up

    the dictionary with a key, and returns a String
  57. Result Total allocated: 1336 bytes (9 objects) allocated memory by

    class ----------------------------------- 928 Hash 368 Array 40 String allocated objects by class ----------------------------------- 4 Array 4 Hash 1 String
  58. I18n Creates Some Hash and Array Objects Per Each I18n.t

    Invocation Especially for manipulating given keys and options
  59. Like This? module I18nCache def self.extended(obj) obj.instance_variable_set :@i18n_cache, {} end

    def translate(*args, **options) cache = @i18n_cache[I18n.locale] ||= {} cache_key = args << options cache[cache_key] || begin result = super cache[cache_key] = result if result result ennd alias t translate end I18n.extend I18nCache
  60. This Monkey-Patch May Not Be Production-Ready Yet I'm aware that

    we at least need to handle `:count` option There may be some more corner cases If this approach is too aggressive, maybe we could monkey-patch `Backend#lookup`
  61. Anyway, Now I18n.t Became Much Cheaper $ env RAILS_ENV=production rails

    r 'I18n.t(:hello); Prof.mem { I18n.t :hello }' Total allocated: 272 bytes (2 objects) allocated memory by class ----------------------------------- 232 Hash 40 Array allocated objects by class ----------------------------------- 1 Array 1 Hash
  62. Allocated Objects by Class allocated objects by class ----------------------------------- 3235

    String 1300 Array 1019 Hash 162 ActiveSupport::SafeBuffer 100 ActiveModel::Attribute::FromDatabase 50 ActiveModel::AttributeSet 50 ActiveModel::LazyAttributeHash 50 Post
  63. So Here's a Super Monkey-Patch That Could Reduce These Objects

    https:/ /github.com/ amatsuda/ lightweight_attributes
  64. lightweight_attributes' Approach AR model has too many features But in

    most cases, what we need is "data transfer object" instances that are "read only" Such objects can be made more lightweight by dropping some heavy features e.g. dirty tracking, type casting
  65. Allocated Objects by Class: with lightweight_attributes allocated objects by class

    ----------------------------------- 3187 String 1293 Array 996 Hash 162 ActiveSupport::SafeBuffer 52 Class 50 LightweightAttributes::AttributeSet 50 Post
  66. Allocated Objects by Class: Before => After 3235 String 1300

    Array 1019 Hash 162 ActiveSupport::SafeBuffer 100 ActiveModel::Attribute::FromDatabase 50 ActiveModel::AttributeSet 50 ActiveModel::LazyAttributeHash 50 Post ! 3187 String 1293 Array 996 Hash 162 ActiveSupport::SafeBuffer 52 Class 50 LightweightAttributes::AttributeSet 50 Post
  67. Result: Total Allocated with lightweight_attributes Total allocated: 580397 bytes (6170

    objects) ↓ Total allocated: 586565 bytes (5981 objects)
  68. Probably Because We're Not Accessing All Fields in This Example

    The default implementation lazily type casts The lightweight_attributes version immediately type casts
  69. What Consumes Memory in the index Action? allocated memory by

    location ----------------------------------- 35032 actionpack/lib/action_controller/metal/url_for.rb:42 35032 actionpack/lib/action_dispatch/routing/route_set.rb:199 35032 actionpack/lib/action_dispatch/routing/route_set.rb:231 27200 rubyconfid/app/views/posts/index.html.erb:19 26000 lightweight_attributes/lib/lightweight_attributes/attribute_set.rb:17 25587 activesupport/lib/active_support/core_ext/string/output_safety.rb:175 25368 actionpack/lib/action_dispatch/journey/visitors.rb:49 23432 actionview/lib/action_view/helpers/url_helper.rb:683 18920 actionpack/lib/action_dispatch/journey/visitors.rb:38
  70. The Monkey-Patches for Inflector def camelize(term, uppercase_first_letter = true) return

    super unless uppercase_first_letter if (cached = InflectorCache.camelize_cache[term]) cached.dup else super ennd def underscore(camel_cased_word) if (cached = InflectorCache.underscore_cache[camel_cased_word]) cached.dup else super ennd
  71. And Another Monkey-Patch That Creates the Cache from Table Name

    module SchemaStatementsExtension def columns(table_name) result = super unless InflectorCache.camelize_cache[table_name] InflectorCache.camelize_cache[table_name] = table_name.camelize end unless InflectorCache.underscore_cache[table_name] InflectorCache.underscore_cache[table_name] = table_name.underscor end result ennd ActiveRecord::ConnectionAdapters::AbstractAdapter.prepend SchemaStatementsExtension
  72. The Monkey-Patch Can Be Something Like This module CachedLayout private

    def _layout(lookup_context, formats) cache = self.class.instance_variable_get :@_layout_cache implied_layout_name = self.class.send(:_implied_layout_name) prefixes = /\blayouts/.match?(implied_layout_name) ? [] : ["layouts"] cache[[implied_layout_name, prefixes, formats]] ||= super ennd
  73. Only if Given `:layout` Is `nil` or String def _write_layout_method

    # :nodoc: super if _layout.nil? || (String === _layout) @_layout_cache = {} prepend CachedLayout ennd
  74. Before Layout Cache allocated memory by file ----------------------------------- 4916 rack-2.0.7/lib/rack/utils.rb

    3839 actionpack/lib/action_dispatch/middleware/static.rb 3821 ruby/lib/logger.rb 3800 actionpack/lib/action_dispatch/http/response.rb 3760 actionview/lib/action_view/lookup_context.rb
  75. After Layout Cache allocated memory by file ----------------------------------- 4884 rack-2.0.7/lib/rack/utils.rb

    3839 actionpack/lib/action_dispatch/middleware/static.rb 3821 ruby/lib/logger.rb 3800 actionpack/lib/action_dispatch/http/response.rb 3688 activesupport/lib/active_support/callbacks.rb 3552 actionpack/lib/action_dispatch/middleware/cookies.rb 3504 activesupport/lib/active_support/hash_with_indifferent_access.rb 3465 activesupport/lib/active_support/json/encoding.rb 3458 actionview/lib/action_view/helpers/tag_helper.rb 3311 activesupport/lib/active_support/message_encryptor.rb 2851 activesupport/lib/active_support/core_ext/string/output_safety.rb 2384 activesupport/lib/active_support/messages/rotator.rb 2344 activesupport/lib/active_support/core_ext/hash/keys.rb 2299 rubyconfid/app/views/layouts/application.html.string 2198 activesupport/lib/active_support/core_ext/object/json.rb 2080 actionview/lib/action_view/lookup_context.rb
  76. turbo_partial `render_partial` takes many parameters So it can't simply be

    cached turbo_partial adds `render_relative` and `render_absolute` features to `render_partial`
  77. Who Are Allocating Objects Now? (post/ok) allocated memory by location

    ----------------------------------- 3451 actionpack/lib/action_dispatch/middleware/static.rb:95 1347 ruby/lib/logger.rb:597 1288 actionpack/lib/action_dispatch/http/response.rb:437 1160 activesupport/lib/active_support/notifications/instrumenter.rb:67 1152 activesupport/lib/active_support/callbacks.rb:405
  78. And What Is This Binding Thing? allocated memory by class

    ----------------------------------- 14440 Hash 9987 String 9688 Array 2520 MatchData 1728 ActionDispatch::Request 1608 Thread::Backtrace 1160 ActiveSupport::HashWithIndifferentAccess 1040 Proc 944 Rack::Utils::HeaderHash 912 Class 512 Regexp 352 Binding
  79. binding.local_variable_ get A magical idiom to get a local variable

    that has a name that conflicts with reserved words
  80. This Doesn't Usually Happen, But Keyword Arguments Can Create This

    def do_something(if: nil, unless: nil) if if ... end if unless ... ennd
  81. But It Turned Out That It Allocates a Binding Object

    I hope we can fix this in Ruby 2.7
  82. HashWithIndifferentAc cess A variety of Hash that treats Symbol keys

    and String keys as the same things Converts all Symbol keys to String internally Abused too much in the Rails internal
  83. Maybe We Can Reduce HashWithIndifferentAccess By defaulting this to be

    Symbol based Thanks to Symbol GC from @nari3 Or by implementing this in C
  84. Today's Benchmark Result Becomes Like This allocated memory by file

    ----------------------------------- 7472 ruby/lib/monitor.rb 4908 rack-2.0.7/lib/rack/utils.rb 3839 actionpack/lib/action_dispatch/middleware/static.rb 3821 ruby/lib/logger.rb 3800 actionpack/lib/action_dispatch/http/response.rb 3688 activesupport/lib/active_support/callbacks.rb 3552 actionpack/lib/action_dispatch/middleware/cookies.rb
  85. Actually I Baked Some of These Silly Monkey-Patches into a

    Gem https:/ /github.com/ amatsuda/speed_king
  86. Wrap Up Rails is a huge software that has no

    obvious bottleneck But we can somehow improve it by micro- optimizations I already pushed so many patches in Rails master, so Rails 6.1 has to be faster than 6.0 After such micro-optimizations, today's benchmark highlighted some performance hotspots
  87. Wrap up (3) Rails is such a fun thing to

    play with! Performance tuning is like a game aiming for a high-score Everybody can play with it for free! Since it's open sourced! I'm waiting for you to join this game!
  88. end