mondrian_redis_segment_cache 0.4.0-java → 0.4.1-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c7637717595cdf7c5447a26dead15bc177106402
-  data.tar.gz: 98088ae79461e0a5b484385275858b35fb58bf0f
+  metadata.gz: c120ed66894ca27037fa5ab85ecd56fd9c57dbd0
+  data.tar.gz: 33fd9bcd2484e75540746890522b771d4e59666c
 SHA512:
-  metadata.gz: 612220016333dfd5b7b667cf48a6effe1dff8ebe2a36ec46dac325f6c8b56c7e8f00df631477ba8be4a395bed529d2227e6ddb7a63acce44f42e03f8fb96cd1f
-  data.tar.gz: 398b873c2906b1738152e52e224a44af99e0450030566ba0c0ca31cc34b79b0a843d54be8f08727154e55f899c125fd1c157895c53f179ebbcd7157e0a8287c9
+  metadata.gz: f7bb09883b38bbe9ecfa321efe3e1eaf41cab8e7b0fbdb71a2b041fed15a1d557a52e7d1eea7391201bf686609c5f850ea7e88d930162206acff19f4779a8e6b
+  data.tar.gz: 4128c1f68b480e8b25c83ccbaab47fbec40cf1500a8af407e9874c7c7fc30dbda66e55d7649e176573343a4356169b8c3f020232de5db7977de6865b5bd1e28d

data/lib/mondrian_redis_segment_cache/cache.rb

@@ -20,8 +20,8 @@ module MondrianRedisSegmentCache
     def initialize(mondrian_redis_connection, new_options = {})
       @mondrian_redis = mondrian_redis_connection
       @options = Marshal.load(Marshal.dump(new_options))
-      @listeners = Set.new
-      @local_cache_set = Set.new
+      @listeners = ::Concurrent::Array.new
+      @local_cache_set = ::Concurrent::Array.new
 
       ##
       # Having a TimerTask reconcile every 5 minutes so the local listeners are eventually consistent with
@@ -39,7 +39,8 @@ module MondrianRedisSegmentCache
     # Public Instance Methods
     #
     def addListener(segment_cache_listener)
-      listeners << segment_cache_listener
+      listeners << segment_cache_listener unless listeners.include?(segment_cache_listener)
+      listeners.uniq!
     end
 
     # returns mondrian.spi.SegmentBody
@@ -78,7 +79,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
+      @local_cache_set << header_base64 unless @local_cache_set.include?(header_base64)
       mondrian_redis.with do |connection|
         connection.sadd(SEGMENT_HEADERS_SET_KEY, header_base64)
       end
@@ -93,7 +94,6 @@ module MondrianRedisSegmentCache
         end
       end
 
-      publish_created_to_listeners(header_base64)
       return ("#{set_success}".upcase == "OK" || set_success == true) # weird polymorphic return ?
     end
 
@@ -113,7 +113,6 @@ module MondrianRedisSegmentCache
         connection.del(header_base64)
       end
 
-      publish_deleted_to_listeners(header_base64)
       return deleted_keys >= 1
     end
 
@@ -189,7 +188,7 @@ module MondrianRedisSegmentCache
     end
 
     def reconcile_local_set_with_redis
-      remote_set = Set.new
+      remote_set = []
 
       mondrian_redis.with do |connection|
         connection.sscan_each(SEGMENT_HEADERS_SET_KEY) do |segment_header_base64|
@@ -197,8 +196,10 @@ module MondrianRedisSegmentCache
         end
       end
 
-      remote_added_keys = remote_set - local_cache_set
-      remote_removed_keys = local_cache_set - remote_set
+      local_copy = []
+      local_cache_set.each { |value| local_copy << value }
+      remote_added_keys = remote_set - local_copy
+      remote_removed_keys = local_copy - remote_set
 
       remote_added_keys.each do |remote_added_key|
         @local_cache_set << remote_added_key
@@ -209,6 +210,8 @@ 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.0"
+  VERSION = "0.4.1"
 end

metadata

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