scribe-rb 2.2.rc4 → 2.2.rc5

data/lib/extras/queued_log_message_handler.rb

@@ -2,7 +2,12 @@ require "thread"
 
 module FacebookService
   class QueuedLogMessageHandler < BaseHandler
-    attr_reader :queue
+    attr_reader   :queue
+
+    # If message_limit is exceeded, the handler will not accept any more
+    # messages from Scribe. The messages will be kept by Scribe and sent
+    # later.
+    attr_accessor :message_limit
 
     def initialize(name)
       super(name)
@@ -11,11 +16,15 @@ module FacebookService
     end
 
     def Log(messages)
-      messages.each do |message|
-        @queue.push(message)
-      end
+      if self.message_limit.nil? || self.queue.size < self.message_limit
+        messages.each do |message|
+          self.queue.push(message)
+        end
 
-      ::ResultCode::OK
+        ::ResultCode::OK
+      else
+        ::ResultCode::TRY_LATER
+      end
     rescue Exception => e
       ::ResultCode::TRY_LATER
     end

data/scribe-rb.gemspec

@@ -3,7 +3,7 @@ $:.push File.expand_path("../lib", __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = "scribe-rb"
-  s.version     = "2.2.rc4"
+  s.version     = "2.2.rc5"
   s.authors     = ["Andy Lindeman"]
   s.email       = ["andy@highgroove.com"]
   s.homepage    = "http://github.com/highgroove/scribe-rb"

data/spec/lib/extras/queued_log_message_handler_spec.rb

@@ -20,4 +20,20 @@ describe FacebookService::QueuedLogMessageHandler do
       subject.Log(["foo"]).should eq(ResultCode::OK)
     end
   end
+
+  context "with message limit" do
+    it "accepts messages and returns ResultCode::OK if the size limit is not exceeded" do
+      subject.message_limit = 1
+
+      subject.Log(["foo"]).should eq(ResultCode::OK)
+      subject.queue.size.should eq(1)
+    end
+
+    it "does not accept messages and returns ResultCode::TRY_LATER if the size limit is exceeded" do
+      subject.message_limit = 0
+
+      subject.Log(["foo"]).should eq(ResultCode::TRY_LATER)
+      subject.queue.size.should be_zero
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: scribe-rb
 version: !ruby/object:Gem::Version
-  version: 2.2.rc4
+  version: 2.2.rc5
   prerelease: 4
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-09-20 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thrift
-  requirement: &2156662740 !ruby/object:Gem::Requirement
+  requirement: &2156647800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 0.7.0
   type: :runtime
   prerelease: false
-  version_requirements: *2156662740
+  version_requirements: *2156647800
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &2156662140 !ruby/object:Gem::Requirement
+  requirement: &2156647060 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2156662140
+  version_requirements: *2156647060
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2156661320 !ruby/object:Gem::Requirement
+  requirement: &2156645840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: 2.6.0
   type: :development
   prerelease: false
-  version_requirements: *2156661320
+  version_requirements: *2156645840
 - !ruby/object:Gem::Dependency
   name: timecop
-  requirement: &2156660580 !ruby/object:Gem::Requirement
+  requirement: &2156645080 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +54,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2156660580
+  version_requirements: *2156645080
 description: Everything you need to build a Ruby producer/client or consumer/server
   of Scribe messages
 email:
@@ -98,7 +98,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -57725399886143313
+      hash: -3771262154631515498
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: