Quick Wins

Six quick and easy techniques that could significantly impact the performance of your Rails application.

Transcript

1. collectiveidea.com Rails Performance “Quick Wins” Steve Richert @laserlemon
2. collectiveidea.com
3. collectiveidea.com
4. collectiveidea.com ✉️
5. collectiveidea.com Demo
6. collectiveidea.com github.com/laserlemon/quick-wins
7. collectiveidea.com ActiveRecord::Schema.define(version: 20140702141655) do ! create_table "messages", force: true do |t| t.integer "sender_id" t.integer "recipient_id" t.string "subject" t.text "body" t.boolean "read", default: false, null: false t.datetime "created_at" t.datetime "updated_at" end ! create_table "users", force: true do |t| t.string "name" t.datetime "created_at" t.datetime "updated_at" t.string "username" t.string "password_digest" end ! end
8. collectiveidea.com class MessagesController < ApplicationController before_action :require_current_user ! def index @messages = current_user.received_messages.new_to_old end ! def mark_read @messages = current_user.received_messages.where(id: params[:message_ids]) ! @messages.each do |message| message.update_attribute(:read, true) end ! redirect_to messages_path end ! private ! def require_current_user unless current_user redirect_to login_path, flash: { alert: "Please log in first." } end end ! def current_user User.find_by(id: cookies[:current_user_id]) end end
9. collectiveidea.com ActiveRecord::Schema.define(version: 20140702141655) do ! create_table "messages", force: true do |t| t.integer "sender_id" t.integer "recipient_id" t.string "subject" t.text "body" t.boolean "read", default: false, null: false t.datetime "created_at" t.datetime "updated_at" end ! create_table "users", force: true do |t| t.string "name" t.datetime "created_at" t.datetime "updated_at" t.string "username" t.string "password_digest" end ! end Y U NO INDEX?!
10. collectiveidea.com ~ English Proverb ✌️✂️☝️
11. collectiveidea.com require "spec_helper" ! describe "Message List Performance", type: :feature do let!(:user) { create(:user) } ! before do create_list(:message, 1_000, recipient: user) log_in(user) end ! it "is fast!" do measure do visit(messages_path) ! 10.times do |i| # Refresh the page twice 2.times { visit(current_path) } ! # Select the first 100 unread messages dom_messages = DOM::Message.all dom_messages.slice(i * 100, 100).each(&:select) ! # Mark the selected messages as read click_button("Mark as Read") end # Rinse, repeat end end end
12. collectiveidea.com 1. Database Indexes
13. collectiveidea.com class AddIndexesToMessages < ActiveRecord::Migration def change add_index :messages, :recipient_id add_index :messages, :created_at end end
14. collectiveidea.com Listing Messages 0 ms 200 ms 400 ms 600 ms 800 ms 1,000 ms 1,200 ms 1,400 ms master database-indexes 955.4 937.2 280.5 279.8 Active Record Views
15. collectiveidea.com Marking Messages 0 ms 10 ms 20 ms 30 ms 40 ms 50 ms 60 ms 70 ms master database-indexes 64.0 58.7 Active Record
16. collectiveidea.com
17. collectiveidea.com 2. Memoization
18. collectiveidea.com class MessagesController < ApplicationController before_action :require_current_user ! def index @messages = current_user.received_messages.new_to_old end ! def mark_read @messages = current_user.received_messages.where(id: params[:message_ids]) ! @messages.each do |message| message.update_attribute(:read, true) end ! redirect_to messages_path end ! private ! def require_current_user unless current_user redirect_to login_path, flash: { alert: "Please log in first." } end end ! def current_user User.find_by(id: cookies[:current_user_id]) end end
19. collectiveidea.com def current_user User.find_by(id: cookies[:current_user_id]) end
20. collectiveidea.com def current_user @current_user ||= User.find_by(id: cookies[:current_user_id]) end
21. collectiveidea.com Listing Messages 0 ms 200 ms 400 ms 600 ms 800 ms 1,000 ms 1,200 ms 1,400 ms master memoization 923.1 937.2 273.1 279.8 Active Record Views
22. collectiveidea.com Marking Messages 0 ms 10 ms 20 ms 30 ms 40 ms 50 ms 60 ms 70 ms master memoization 58.3 58.7 Active Record
23. collectiveidea.com
24. collectiveidea.com 3. Eager Loading
25. collectiveidea.com <% @messages.each do |message| %> <tr class="<%= message.read? ? "read" : "unread" %> message"> <td class=“action"> <%= check_box_tag("message_ids[]", message.id) unless message.read? %> </td> <td class="from"><%= message.sender.name %></td> <td class="to"><%= message.recipient.name %></td> <td class="subject"><%= message.subject %></td> </tr> <% end %>
26. collectiveidea.com n + 1
27. collectiveidea.com
28. collectiveidea.com class MessagesController < ApplicationController before_action :require_current_user ! def index @messages = current_user.received_messages.new_to_old end ! def mark_read @messages = current_user.received_messages.where(id: params[:message_ids]) ! @messages.each do |message| message.update_attribute(:read, true) end ! redirect_to messages_path end ! private ! def require_current_user unless current_user redirect_to login_path, flash: { alert: "Please log in first." } end end ! def current_user User.find_by(id: cookies[:current_user_id]) end end
29. collectiveidea.com def index @messages = current_user.received_messages.new_to_old end
30. collectiveidea.com def index @messages = current_user.received_messages.new_to_old.includes(:sender) end
31. collectiveidea.com Listing Messages 0 ms 200 ms 400 ms 600 ms 800 ms 1,000 ms 1,200 ms 1,400 ms master eager-loading 631.6 937.2 153.3 279.8 Active Record Views
32. collectiveidea.com
33. collectiveidea.com 4. Inverse Association
34. collectiveidea.com <% @messages.each do |message| %> <tr class="<%= message.read? ? "read" : "unread" %> message"> <td class=“action"> <%= check_box_tag("message_ids[]", message.id) unless message.read? %> </td> <td class="from"><%= message.sender.name %></td> <td class="to"><%= message.recipient.name %></td> <td class="subject"><%= message.subject %></td> </tr> <% end %>
35. collectiveidea.com class User < ActiveRecord::Base has_many :sent_messages, class_name: "Message", foreign_key: "sender_id" ! has_many :received_messages, class_name: "Message", foreign_key: "recipient_id" end
36. collectiveidea.com class User < ActiveRecord::Base has_many :sent_messages, class_name: "Message", foreign_key: "sender_id" ! has_many :received_messages, class_name: "Message", foreign_key: "recipient_id", inverse_of: :recipient end
37. collectiveidea.com Listing Messages 0 ms 200 ms 400 ms 600 ms 800 ms 1,000 ms 1,200 ms 1,400 ms master inverse-association 563.8 937.2 141.5 279.8 Active Record Views
38. collectiveidea.com ❗️
39. collectiveidea.com 5. Fragment Caching
40. collectiveidea.com <% @messages.each do |message| %> ! <tr class="<%= message.read? ? "read" : "unread" %> message"> <td class=“action"> <%= check_box_tag("message_ids[]", message.id) unless message.read? %> </td> <td class="from"><%= message.sender.name %></td> <td class="to"><%= message.recipient.name %></td> <td class="subject"><%= message.subject %></td> </tr> ! <% end %>
41. collectiveidea.com <% @messages.each do |message| %> <% cache message do %> <tr class="<%= message.read? ? "read" : "unread" %> message"> <td class=“action"> <%= check_box_tag("message_ids[]", message.id) unless message.read? %> </td> <td class="from"><%= message.sender.name %></td> <td class="to"><%= message.recipient.name %></td> <td class="subject"><%= message.subject %></td> </tr> <% end %> <% end %>
42. collectiveidea.com Listing Messages 0 ms 200 ms 400 ms 600 ms 800 ms 1,000 ms 1,200 ms 1,400 ms master fragment-caching 592.1 937.2 32.6 279.8 Active Record Views
43. collectiveidea.com ‼️
44. collectiveidea.com 6. Bulk Update
45. collectiveidea.com class MessagesController < ApplicationController before_action :require_current_user ! def index @messages = current_user.received_messages.new_to_old end ! def mark_read @messages = current_user.received_messages.where(id: params[:message_ids]) ! @messages.each do |message| message.update_attribute(:read, true) end ! redirect_to messages_path end ! private ! def require_current_user unless current_user redirect_to login_path, flash: { alert: "Please log in first." } end end ! def current_user User.find_by(id: cookies[:current_user_id]) end end
46. collectiveidea.com @messages.each do |message| message.update_attribute(:read, true) end
47. collectiveidea.com ! @messages.update_all(read: true, updated_at: Time.current)
48. collectiveidea.com Marking Messages 0 ms 10 ms 20 ms 30 ms 40 ms 50 ms 60 ms 70 ms master bulk-update 4.5 58.7 Active Record
49. collectiveidea.com ✖️
50. collectiveidea.com Combined Results
51. collectiveidea.com Listing Messages 0 ms 200 ms 400 ms 600 ms 800 ms 1,000 ms 1,200 ms 1,400 ms master fast 483.2 937.2 13.0 279.8 Active Record Views
52. collectiveidea.com Marking Messages 0 ms 10 ms 20 ms 30 ms 40 ms 50 ms 60 ms 70 ms master fast 5.9 58.7 Active Record
53. collectiveidea.com Thank you❗️
54. collectiveidea.com Questions❓ • github.com/laserlemon/quick-wins • speakerdeck.com/laserlemon/quick-wins • collectiveidea.com • twitter.com/laserlemon