Merge pull request #433 from iFixit/performance-improvements

Huge Performance Improvements

Merge pull request #433 from iFixit/performance-improvements
Huge Performance Improvements
Shuky Dvir
2 parents d89230cf f438aef9
Showing 6 changed files with 9 additions and 2 deletions Show diff stats
app/controllers/api/v1/stats_controller.rb
app/models/app.rb
config/application.rb
config/mongoid.example.yml
config/mongoid.mongohq.yml
config/mongoid.mongolab.yml
@@ -13,7 +13,7 @@ class Api::V1::StatsController &lt; ApplicationController
     stats = {
       :name => @app.name,
       :last_error_time => @last_error_time,
-      :unresolved_errors => @app.problems.unresolved.count
+      :unresolved_errors => @app.unresolved_count
     }
  
     respond_to do |format|
 class App
+  include Comparable
   include Mongoid::Document
   include Mongoid::Timestamps
-  include Comparable
  
   field :name, :type => String
   field :api_key
@@ -48,6 +48,9 @@ module Errbit
       g.fixture_replacement :fabrication
     end
  
+    # Enable the mongoid identity map for performance
+    Mongoid.identity_map_enabled = true
+
     # IssueTracker subclasses use inheritance, so preloading models provides querying consistency in dev mode.
     config.mongoid.preload_models = true
  
@@ -9,6 +9,8 @@
  
 defaults: &defaults
   host: localhost
+  identity_map_enabled: true
+
   # slaves:
   #   - host: slave1.local
   #     port: 27018
@@ -6,3 +6,4 @@
  
 production:
   uri: <%= ENV['MONGOHQ_URL'] %>
+  identity_map_enabled: true
@@ -6,3 +6,4 @@
  
 production:
   uri: <%= ENV['MONGOLAB_URI'] %>
+  identity_map_enabled: true
...	...	@@ -13,7 +13,7 @@ class Api::V1::StatsController < ApplicationController
13	13	stats = {
14	14	:name => @app.name,
15	15	:last_error_time => @last_error_time,
16		- :unresolved_errors => @app.problems.unresolved.count
	16	+ :unresolved_errors => @app.unresolved_count
17	17	}
18	18
19	19	respond_to do \|format\|
...	...
1	1	class App
	2	+ include Comparable
2	3	include Mongoid::Document
3	4	include Mongoid::Timestamps
4		- include Comparable
5	5
6	6	field :name, :type => String
7	7	field :api_key
...	...
...	...	@@ -48,6 +48,9 @@ module Errbit
48	48	g.fixture_replacement :fabrication
49	49	end
50	50
	51	+ # Enable the mongoid identity map for performance
	52	+ Mongoid.identity_map_enabled = true
	53	+
51	54	# IssueTracker subclasses use inheritance, so preloading models provides querying consistency in dev mode.
52	55	config.mongoid.preload_models = true
53	56
...	...
...	...	@@ -9,6 +9,8 @@
9	9
10	10	defaults: &defaults
11	11	host: localhost
	12	+ identity_map_enabled: true
	13	+
12	14	# slaves:
13	15	# - host: slave1.local
14	16	# port: 27018
...	...
...	...	@@ -6,3 +6,4 @@
6	6
7	7	production:
8	8	uri: <%= ENV['MONGOHQ_URL'] %>
	9	+ identity_map_enabled: true
...	...
...	...	@@ -6,3 +6,4 @@
6	6
7	7	production:
8	8	uri: <%= ENV['MONGOLAB_URI'] %>
	9	+ identity_map_enabled: true
...	...