Overwriting OplogHandle

@theodorDiaconu Have you ever considered overwriting OplogHandle class from Meteor’s mongo package?

https://github.com/meteor/meteor/blob/d93c021c896029d774ccecc7241ff20ec4045568/packages/mongo/oplog_tailing.js

I had a quick look and it seems like all the logic specific to mongo oplog tailing is already abstracted to this class.

It might be an interesting alternative to make the package smaller and more tightly bound with observer mechanisms in Meteor core.

I’d be interested to hear your thoughts on this matter.

[simonbelanger]

Yes, we’re still using the package in production. As I speak, we’re running on 10 quad containers for 6000 connections. Before the integration of the package, this number would have been around 30. Subscription response time is steady under 250 ms, 200ms for method.

An interesting fact is that critical metric switched from CPU to memory with the integration of the package. So we were able to save a couple containers just by bumping up the max-old-space-size setting at 3500, CPU still chilling at 50% usage.

[theodorDiaconu]

@simonbelanger yep that is expected since we have a mirror image for every pubsub in the observableCollection, maybe in the future we’re gonna resolve that as well. Very glad it’s a success story for you.