[GitHub] nifi issue #3135: [NIFI-5795] RedisDistributedMapCacheClientService put miss...

2018-12-03 Thread pvillard31
Github user pvillard31 commented on the issue:

https://github.com/apache/nifi/pull/3135
  
Confirmed the reported issue and confirmed the fix. Merging to master, 
thanks @luup2k 


---


[GitHub] nifi issue #3135: [NIFI-5795] RedisDistributedMapCacheClientService put miss...

2018-11-21 Thread pvillard31
Github user pvillard31 commented on the issue:

https://github.com/apache/nifi/pull/3135
  
Agree, I'd prefer to merge this one first, and then take care of the other 
one.


---


[GitHub] nifi issue #3135: [NIFI-5795] RedisDistributedMapCacheClientService put miss...

2018-11-21 Thread zenfenan
Github user zenfenan commented on the issue:

https://github.com/apache/nifi/pull/3135
  
@pvillard31 I wasn't able to. What do you suggest? We'll close this one and 
cover the objective with #3176 ? or we would merge this in once we verify the 
changes and ask @javajefe to rebase #3176? The latter feels like a better one.. 
since this is the first contribution by @luup2k for NiFi


---


[GitHub] nifi issue #3135: [NIFI-5795] RedisDistributedMapCacheClientService put miss...

2018-11-19 Thread pvillard31
Github user pvillard31 commented on the issue:

https://github.com/apache/nifi/pull/3135
  
@zenfenan - did you make progress on this one? I see this change has also 
been added in #3176 


---