plugin-QueuedTracking
plugin-QueuedTracking copied to clipboard
matomo-org
Multi Tenant suggestion
This is a suggestion Multi Tenant. I'm working with a multi tenant setup as described here in the Matomo documentation. The documentation describes how split the mysql database. Having multiple redis-server as backends can be complicated instead. It could be possibile to use multiple databases in redis, but performance can become awful when period flush to disk is active. Adding a suffix to the list key with the Matomo hostname picked from the $trusted_hosts config variable can be a solution, avoiding conflicts across instances.
@AltamashShaikh Do you know why this PR was never merged? Did we simply not notice that it had been created? Do you see a reason not to merge this or should we wait till after Matomo 5.0.0?
@snake14 Dunno why this was never merged, I think we missed it and just adding braces with if is my suggestion to this PR
@snake14 This would require migration too right ? Since we updated the key
@snake14 This would require migration too right ? Since we updated the key
Good call @AltamashShaikh . Since it would be looking for a different key, we would probably want a migration for existing installations. I'll mark this PR for needing to be moved to 5.x.