search

quavedev / meteor-packages

8 stars 2 forks source link

redis-oplog changes? #3

Closed brianlukoff closed 1 month ago

brianlukoff commented 2 months ago

I've been experiencing some hard-to-reproduce issues in production with rc.2 of the MCP version of redis-oplog (which I believe is from this PR), and wondering what changes were made to the quave:redis-oplog version -- is this version working well in production?

filipenevola commented 2 months ago

As far as we know yes @brianlukoff.

We are aware of some problems when Monti APM agent is also included but besides that it is working fine.

@matheusccastroo any other issues that you can mention here?

brianlukoff commented 2 months ago

Thanks for the info! What are the issues you've run into with Monti? We started using it with Monti and haven't had any issues (although since we are on the 3.0 Monti beta we don't get any redis-oplog stats in Monti).

matheusccastroo commented 1 month ago

@brianlukoff actually it wasn't really related to redis-oplog. We faced some issues where the async stack traces were getting mixed, but this is actually an issue on monti itself - so yeah, the redis-oplog is running fine.

filipenevola commented 1 month ago

Closing this issue as Matheus confirmed our redis-oplog version works greatly in our client.

Thanks ;)

zodern commented 1 month ago

@matheusccastroo would you be able to send me the details on the issue with the Monti APM agent?