Move GetAllByUserId to IOnlineClientManager instead of implementing as a static extension

The default implementation of IOnlineClientManager use a ConcurrentDictionary<string, IOnlineClient> to store the relations between userId and connectionId.

It has problem when sites are deploy to two or more servers, or just one server but enable webgarden in iis (has multi process).

So, I wrote a RedisOnlineClientManager to replace OnlineClientManager, but found it may cause performance problem when someone call GetAllByUserId in OnlineClientManagerExtensions through GetAllClients.

Above all, I think It’s a better practice to define GetAllByUserId in IOnlineClientManager, and leave the how-to-query to the real store mechanism (not a ConcurrentDictionary).

[personball]

@gunpal5 https://github.com/personball/abplus/blob/master/src/Abplus.Web.SignalR/RealTime/RedisOnlineClientManager.cs

[jmhinnen]

@personball I think I see the issue with using the Cache. It’s the sliding expiration policy cache that may get in the way. I think your implementation is much better than what I was working towards. Never mind on my question above.