Removed , but not the table. Record all in httpmulti success

app/models/job/http_multi.rb

@@ -37,6 +37,9 @@ module Job
         request = Request.new(url, OPTS.merge(:params => {:xml => CGI::escape(xml)}))
 
         request.on_complete do |response|
+          # Save the reference to the pod to the database if not already present
+          Pod.find_or_create_by_url(response.effective_url)
+
           if response.code >= 300 && response.code < 400
             if response.headers_hash['Location'] == response.request.url.sub('http://', 'https://')
               location = URI.parse(response.headers_hash['Location'])
@@ -48,10 +51,7 @@ module Job
             end
           end
           unless response.success?
-            pod = Pod.find_or_create_by_url(response.effective_url)
-            log_line = "event=http_multi_fail sender_id=#{user_id} recipient_id=#{person.id} url=#{response.effective_url} response_code='#{response.code}'"
-            Rails.logger.info(log_line)
-            pod.pod_stats.create(:error_message => log_line, :person_id => person.id, :error_code => response.code.to_i)
+            Rails.logger.info("event=http_multi_fail sender_id=#{user_id} recipient_id=#{person.id} url=#{response.effective_url} response_code='#{response.code}'")
             failed_request_people << person.id
           end
         end

app/models/pod.rb

 class Pod < ActiveRecord::Base
-  has_many :pod_stats
-
   def self.find_or_create_by_url(url)
     u = URI.parse(url)
     pod = self.find_or_initialize_by_host(u.host)

app/models/pod_stat.rb

-class PodStat < ActiveRecord::Base
-  belongs_to :pod
-  
-end

db/migrate/20110830170929_remove_pod_stats_table.rb

+class RemovePodStatsTable < ActiveRecord::Migration
+  def self.up
+    drop_table :pod_stats
+  end
+
+  def self.down
+    create_table :pod_stats do |t|
+      t.integer :error_code
+      t.integer :person_id
+      t.text :error_message
+      t.integer :pod_id
+
+      t.timestamps
+    end
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20110818212541) do
+ActiveRecord::Schema.define(:version => 20110830170929) do
 
   create_table "aspect_memberships", :force => true do |t|
     t.integer  "aspect_id",  :null => false
@@ -236,15 +236,6 @@ ActiveRecord::Schema.define(:version => 20110818212541) do
   add_index "people", ["guid"], :name => "index_people_on_guid", :unique => true
   add_index "people", ["owner_id"], :name => "index_people_on_owner_id", :unique => true
 
-  create_table "pod_stats", :force => true do |t|
-    t.integer  "error_code"
-    t.integer  "person_id"
-    t.text     "error_message"
-    t.integer  "pod_id"
-    t.datetime "created_at"
-    t.datetime "updated_at"
-  end
-
   create_table "pods", :force => true do |t|
     t.string   "host"
     t.boolean  "ssl"

spec/models/pod_spec.rb

 require 'spec_helper'
 
 describe Pod do
-
-  it 'has many pod_stats' do
-    Pod.new.pod_stats.should be_empty
-  end
   describe '.find_or_create_by_url' do
     it 'takes a url, and makes one by host' do
       pod = Pod.find_or_create_by_url('https://joindiaspora.com/maxwell')

spec/models/pod_stat_spec.rb

-require 'spec_helper'
-
-describe PodStat do
-  pending "add some examples to (or delete) #{__FILE__}"
-end