splitclient-rb 5.0.2.pre.rc1-java → 5.0.3-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fbae000c8e4e3ec9bd6cf3aa3b836e91b71df014
-  data.tar.gz: faf5514108c7bda89d8e1c0b8148c8ad09f9cb55
+  metadata.gz: 3d25dfd36c9ddd3f35ae8be7bfd1d0dc4e6de872
+  data.tar.gz: d78ae6bd4d7161f707ea3559f1c05236de2362f1
 SHA512:
-  metadata.gz: edfbb0409a4b6c9e0446ca58e1ae39ae0fbc6f8119a887122cff5e93218803f2d37dbda0e82502c3c327f829144e219ad35791d9bfe09f4bdb92dfb4d1383f2b
-  data.tar.gz: 6580f82bfaa89ae94c8d5c00b2cca211a8eda81cc0e29dd26ef7f7a3d183ed2e436dbccd85c2690ce8299494cbb87c480ac235dac671e7115a91ce6907c6b503
+  metadata.gz: bff71d7dc6a4efe9527aada4f34974d8138e0d84a54f308dfb06772c3737b6f1550b8c19094caf35da1b0c34e012de7a8a5220f3ad69ab0a28d3227a48d6f4c3
+  data.tar.gz: 8204a8a2d9a986f726798506ad99172f505660cf39732999a0dea59c637122f3553983d2a0204f1ff33a237e4f0b1bf7c355938a8ac0fe903d6eb8208d612e5d

data/CHANGES.txt

@@ -1,3 +1,6 @@
+5.0.3 (August 13th, 2018)
+- Add `impressions_bulk_size` option to set the max number of impressions to be sent to the Split backend on each post.
+
 5.0.2 (July 31st, 2018)
 - Prevents the impression thread from being started if a listener is not in place
 

data/Detailed-README.md

@@ -136,7 +136,9 @@ split_client.get_treatment('user_id','feature_name', attr: 'val')
 
 **impressions_refresh_rate** : The SDK sends information on who got what treatment at what time back to Split servers to power analytics. This parameter controls how often this data is sent to Split servers in seconds
 
-**impressions_queue_size** : The size of the impressions queue in case of `cache_adapter == :memory` and the size impressions batch to be fetched from Redis in case of `cache_adapter == :redis`. Use `-1` to disable impressions.
+**impressions_queue_size** : The size of the impressions queue in case of `cache_adapter == :memory`.
+
+**impressions_bulk_size** : Max number of impressions to be sent to the backend on each post. Defaults to `impressions_queue_size`.
 
 *default value* = `60`
 

data/NEWS

@@ -1,3 +1,9 @@
+5.0.3
+
+Creates a new configuration parameter (impressions_bulk_size) to manage the max number of impressions sent to the Split backend on each post.
+Changes impressions_queue_size so that it manages only the queue size in the memory adapter.
+A backwards compatibility fail safe is included for users of previous SDK versions.
+
 5.0.2
 
 Turn Impression Listener into an optional SDK feature.

data/lib/splitclient-rb/cache/adapters/memory_adapters/queue_adapter.rb

@@ -24,19 +24,21 @@ module SplitIoClient
             @current_size.increment
           end
 
-          # Get all items from the queue
           def clear
-            items = []
+            get_batch(@current_size.value)
+          end
 
-            loop do
+          def get_batch(size)
+            items = []
+            size.times do
               items << @queue.pop(true)
-
               @current_size.decrement
             end
-
-          rescue ThreadError
+            items
+            rescue ThreadError
             items
           end
+
         end
       end
     end

data/lib/splitclient-rb/cache/repositories/impressions/memory_repository.rb

@@ -3,6 +3,7 @@ module SplitIoClient
     module Repositories
       module Impressions
         class MemoryRepository
+
           def initialize(adapter, config)
             @adapter = adapter
             @config = config
@@ -18,23 +19,25 @@ module SplitIoClient
             end
           end
 
-          def add_bulk(key, bucketing_key, treatments_labels_change_numbers, time)
-            treatments_labels_change_numbers.each do |split_name, treatment_label_number|
+          def add_bulk(key, bucketing_key, treatments, time)
+            treatments.each do |split_name, treatment|
               add(
                 split_name,
                 'keyName' => key,
                 'bucketingKey' => bucketing_key,
-                'treatment' => treatment_label_number[:treatment],
-                'label' => @config.labels_enabled ? treatment_label_number[:label] : nil,
-                'changeNumber' => treatment_label_number[:change_number],
+                'treatment' => treatment[:treatment],
+                'label' => @config.labels_enabled ? treatment[:label] : nil,
+                'changeNumber' => treatment[:change_number],
                 'time' => time
               )
             end
           end
 
-          # Get everything from the queue and leave it empty
-          def clear
-            @adapter.clear.map { |impression| impression.update(ip: @config.machine_ip) }
+          def get_batch
+            return [] if @config.impressions_bulk_size == 0
+            @adapter.get_batch(@config.impressions_bulk_size).map do |impression|
+              impression.update(ip: @config.machine_ip)
+            end
           end
 
           private

data/lib/splitclient-rb/cache/repositories/impressions/redis_repository.rb

@@ -3,7 +3,6 @@ module SplitIoClient
     module Repositories
       module Impressions
         class RedisRepository < Repository
-          IMPRESSIONS_SLICE = 1000
 
           def initialize(adapter, config)
             @adapter = adapter
@@ -18,25 +17,23 @@ module SplitIoClient
             )
           end
 
-          def add_bulk(key, bucketing_key, treatments_labels_change_numbers, time)
+          def add_bulk(key, bucketing_key, treatments, time)
             @adapter.redis.pipelined do
-              treatments_labels_change_numbers.each_slice(IMPRESSIONS_SLICE) do |treatments_labels_change_numbers_slice|
-                treatments_labels_change_numbers_slice.each do |split_name, treatment_label_change_number|
-                  add(split_name,
-                      'keyName' => key,
-                      'bucketingKey' => bucketing_key,
-                      'treatment' => treatment_label_change_number[:treatment],
-                      'label' => @config.labels_enabled ? treatment_label_change_number[:label] : nil,
-                      'changeNumber' => treatment_label_change_number[:change_number],
-                      'time' => time)
-                end
+              treatments.each do |split_name, treatment|
+                add(split_name,
+                    'keyName' => key,
+                    'bucketingKey' => bucketing_key,
+                    'treatment' => treatment[:treatment],
+                    'label' => @config.labels_enabled ? treatment[:label] : nil,
+                    'changeNumber' => treatment[:change_number],
+                    'time' => time)
               end
             end
           end
 
-          # Get random impressions from redis in batches of size @config.impressions_queue_size,
+          # Get random impressions from redis in batches of size @config.impressions_bulk_size,
           # delete fetched impressions afterwards
-          def clear
+          def get_batch
             impressions = impression_keys.each_with_object([]) do |key, memo|
               ip = key.split('/')[-2] # 'prefix/sdk_lang/ip/impressions.name' -> ip
               if ip.nil?
@@ -44,7 +41,7 @@ module SplitIoClient
                 next
               end
               split_name = key.split('.').last
-              members = @adapter.random_set_elements(key, @config.impressions_queue_size)
+              members = @adapter.random_set_elements(key, @config.impressions_bulk_size)
               members.each do |impression|
                 parsed_impression = JSON.parse(impression)
 
@@ -61,6 +58,8 @@ module SplitIoClient
             impressions
           rescue StandardError => e
             @config.logger.error("Exception while clearing impressions cache: #{e}")
+
+            []
           end
 
           private
@@ -70,6 +69,8 @@ module SplitIoClient
             @adapter.find_sets_by_prefix("#{@config.redis_namespace}/*/impressions.*")
           rescue StandardError => e
             @config.logger.error("Exception while fetching impression_keys: #{e}")
+
+            []
           end
         end
       end

data/lib/splitclient-rb/cache/repositories/impressions_repository.rb

@@ -4,7 +4,7 @@ module SplitIoClient
       # Repository which forwards impressions interface to the selected adapter
       class ImpressionsRepository < Repository
         extend Forwardable
-        def_delegators :@adapter, :add, :add_bulk, :clear, :empty?
+        def_delegators :@adapter, :add, :add_bulk, :get_batch, :empty?
 
         def initialize(adapter, config)
           @config = config

data/lib/splitclient-rb/cache/senders/impressions_formatter.rb

@@ -7,7 +7,7 @@ module SplitIoClient
         end
 
         def call(raw_impressions)
-          impressions = raw_impressions ? raw_impressions : @impressions_repository.clear
+          impressions = raw_impressions ? raw_impressions : @impressions_repository.get_batch
           formatted_impressions = []
           filtered_impressions = filter_impressions(impressions)
 

data/lib/splitclient-rb/cache/senders/impressions_sender.rb

@@ -9,8 +9,7 @@ module SplitIoClient
         end
 
         def call
-          # Disable impressions if @config.impressions_queue_size == -1
-          if @config.impressions_queue_size < 0
+          if @config.disable_impressions
             @config.logger.info('Disabling impressions service by config')
             return
           end

data/lib/splitclient-rb/clients/split_client.rb

@@ -27,7 +27,7 @@ module SplitIoClient
           memo.merge!(name => get_treatment(key, name, attributes, data, false, true, evaluator))
         end
 
-      if @config.impressions_queue_size > 0
+      unless @config.disable_impressions
         time = (Time.now.to_f * 1000.0).to_i
         @impressions_repository.add_bulk(
           matching_key, bucketing_key, treatments_labels_change_numbers, time
@@ -103,7 +103,6 @@ module SplitIoClient
 
       begin
         latency = (Time.now - start) * 1000.0
-        # Disable impressions if @config.impressions_queue_size == -1
         split && store_impression(split_name, matching_key, bucketing_key, treatment_data, store_impressions, attributes)
 
         # Measure
@@ -145,7 +144,7 @@ module SplitIoClient
     def store_impression(split_name, matching_key, bucketing_key, treatment, store_impressions, attributes)
       time = (Time.now.to_f * 1000.0).to_i
 
-      return unless @config.impressions_queue_size > 0 && store_impressions
+      return if @config.disable_impressions || !store_impressions
 
       @impressions_repository.add(split_name,
         'keyName' => matching_key,

data/lib/splitclient-rb/split_config.rb

@@ -21,9 +21,9 @@ module SplitIoClient
     # @option opts [Int] :impressions_refresh_rate
     # @option opts [Object] :logger a logger to user for messages from the client. Defaults to stdout
     # @option opts [Boolean] :debug_enabled (false) The value for the debug flag
-    # @option opts [Int] :impressions_queue_size how big the impressions queue is before dropping impressions. -1 to disable it.
+    # @option opts [Int] :impressions_queue_size Size of the impressions queue in the memory repository. Once reached, newer impressions will be dropped
+    # @option opts [Int] :impressions_bulk_size Max number of impressions to be sent to the backend on each post
     # @option opts [#log] :impression_listener this object will capture all impressions and process them through `#log`
-    #
     # @return [type] SplitConfig with configuration options
     def initialize(opts = {})
       @base_uri = (opts[:base_uri] || SplitConfig.default_base_uri).chomp('/')
@@ -45,6 +45,10 @@ module SplitIoClient
       @impressions_adapter = SplitConfig.init_cache_adapter(
         opts[:cache_adapter] || SplitConfig.default_cache_adapter, :queue_adapter, @redis_url, @impressions_queue_size
       )
+      #Safeguard for users of older SDK versions.
+      @disable_impressions = @impressions_queue_size == -1
+      #Safeguard for users of older SDK versions.
+      @impressions_bulk_size = opts[:impressions_bulk_size] || @impressions_queue_size > 0 ? @impressions_queue_size : 0
 
       @metrics_adapter = SplitConfig.init_cache_adapter(
         opts[:cache_adapter] || SplitConfig.default_cache_adapter, :map_adapter, @redis_url, false
@@ -179,6 +183,8 @@ module SplitIoClient
     #
     # @return [Integer]
     attr_reader :impressions_queue_size
+    attr_reader :impressions_bulk_size
+    attr_reader :disable_impressions
 
     attr_reader :redis_url
     attr_reader :redis_namespace

data/lib/splitclient-rb/version.rb

@@ -1,3 +1,3 @@
 module SplitIoClient
-  VERSION = '5.0.2.pre.rc1'
+  VERSION = '5.0.3'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: splitclient-rb
 version: !ruby/object:Gem::Version
-  version: 5.0.2.pre.rc1
+  version: 5.0.3
 platform: java
 authors:
 - Split Software
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2018-07-31 00:00:00.000000000 Z
+date: 2018-08-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -309,9 +309,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project:
 rubygems_version: 2.6.14