You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The comments clearly state that you guys at rockset don't use it in prod. Fair enough. But if it is still recommended to use and has an option to turn it on, then it should at least do something.
I've seen the purger recommended in different places:
The comments clearly state that you guys at rockset don't use it in prod. Fair enough. But if it is still recommended to use and has an option to turn it on, then it should at least do something.
Right now, it is both broken and not doing anything (see that commented code, also here).
Those TODOs are 7 years old. The option's documentation says that it is
true
by default but fortunately it is actuallyfalse
.For those reasons, I recommend deleting that option along the code because that's just dead code at this point.
The text was updated successfully, but these errors were encountered: