mondrian_redis_segment_cache 0.4.1-java → 0.4.2-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c120ed66894ca27037fa5ab85ecd56fd9c57dbd0
-  data.tar.gz: 33fd9bcd2484e75540746890522b771d4e59666c
+  metadata.gz: 0c5b0fc11676333a22bf8d22f1c4ab0df228cb88
+  data.tar.gz: 15b9f22447a307040ad73ac05d5e4deae2db36da
 SHA512:
-  metadata.gz: f7bb09883b38bbe9ecfa321efe3e1eaf41cab8e7b0fbdb71a2b041fed15a1d557a52e7d1eea7391201bf686609c5f850ea7e88d930162206acff19f4779a8e6b
-  data.tar.gz: 4128c1f68b480e8b25c83ccbaab47fbec40cf1500a8af407e9874c7c7fc30dbda66e55d7649e176573343a4356169b8c3f020232de5db7977de6865b5bd1e28d
+  metadata.gz: f6e90f71c3d5e5331f4c20470bd556e53797103d28bd054bd3265f7d5ce2f87a31ac90e3f6e5e77fc4d3e474ed9e81238d3bbd7241f8df5e4154685f51df691d
+  data.tar.gz: 20e05ea1857664127c1889fd38360b37b648b8ccca84e318f0fca87ab09e3dbb281174d5a0047a102c6204dbe8162af0f1d0421d1e4976698c1f265a1e5fc561

data/lib/mondrian_redis_segment_cache/cache.rb

@@ -1,9 +1,16 @@
 require 'redis'
 require 'concurrent'
+require 'jruby/synchronized'
 require 'mondrian_redis_segment_cache/created_event'
 require 'mondrian_redis_segment_cache/deleted_event'
+require 'set'
 
 module MondrianRedisSegmentCache
+
+  class SynchronizedSet < ::Set
+    include ::JRuby::Synchronized
+  end
+
   class Cache
     include Java::MondrianSpi::SegmentCache
 
@@ -20,14 +27,14 @@ module MondrianRedisSegmentCache
     def initialize(mondrian_redis_connection, new_options = {})
       @mondrian_redis = mondrian_redis_connection
       @options = Marshal.load(Marshal.dump(new_options))
-      @listeners = ::Concurrent::Array.new
-      @local_cache_set = ::Concurrent::Array.new
+      @listeners = SynchronizedSet.new
+      @local_cache_set = SynchronizedSet.new
 
       ##
-      # Having a TimerTask reconcile every 5 minutes so the local listeners are eventually consistent with
+      # Having a TimerTask reconcile every 6 minutes so the local listeners are eventually consistent with
       # respect to what is in the cache and what has been done .... allows us to get rid of the event
-      # subscribers in the redis API ... consider the job to have timed out after 45 seconds
-      @reconcile_task = ::Concurrent::TimerTask.new(:execution_interval => 360, :timeout_interval => 45) do
+      # subscribers in the redis API ... consider the job to have timed out after 60 seconds
+      @reconcile_task = ::Concurrent::TimerTask.new(:execution_interval => 360, :timeout_interval => 60) do
         reload
       end
 
@@ -39,8 +46,7 @@ module MondrianRedisSegmentCache
     # Public Instance Methods
     #
     def addListener(segment_cache_listener)
-      listeners << segment_cache_listener unless listeners.include?(segment_cache_listener)
-      listeners.uniq!
+      listeners << segment_cache_listener
     end
 
     # returns mondrian.spi.SegmentBody
@@ -79,7 +85,7 @@ module MondrianRedisSegmentCache
       segment_header.getDescription # Hazel adapter says this affects serialization
       header_base64 = segment_header_to_base64(segment_header)
       body_base64 = segment_body_to_base64(segment_body)
-      @local_cache_set << header_base64 unless @local_cache_set.include?(header_base64)
+      @local_cache_set << header_base64
       mondrian_redis.with do |connection|
         connection.sadd(SEGMENT_HEADERS_SET_KEY, header_base64)
       end
@@ -210,8 +216,6 @@ module MondrianRedisSegmentCache
         @local_cache_set.delete(remote_removed_key)
         publish_deleted_to_listeners(remote_removed_key)
       end
-
-      @local_cache_set.uniq!
     end
 
     def reconcile_set_and_keys

data/lib/mondrian_redis_segment_cache/version.rb

@@ -1,3 +1,3 @@
 module MondrianRedisSegmentCache
-  VERSION = "0.4.1"
+  VERSION = "0.4.2"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mondrian_redis_segment_cache
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.4.2
 platform: java
 authors:
 - Brandon Dewitt
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-01-31 00:00:00.000000000 Z
+date: 2017-02-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement