-
Notifications
You must be signed in to change notification settings - Fork 576
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RJS-2760: Adding Realm.shutdown()
and fixing clean exit tests
#6571
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great debugging session 🥳
@@ -0,0 +1,27 @@ | |||
//////////////////////////////////////////////////////////////////////////// | |||
// | |||
// Copyright 2022 Realm Inc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// Copyright 2022 Realm Inc. | |
// Copyright 2024 Realm Inc. |
a2e3900
to
7e5980f
Compare
7e5980f
to
e4eaaa9
Compare
Rebased on |
Coverage test is still crashing (#6589) but that's not new - I'll go ahead and merge with it. |
What, How & Why?
This closes #6555, #5781, #4535, #6096, #6357, #4530, #3616 by adding a static
Realm.shutdown
method, which closes all Realms, cancels all pendingRealm.open
calls, clears internal caches, resets the logger and collects garbage.☑️ ToDos
Compatibility
label is updated or copied from previous entryCOMPATIBILITY.md
package.json
s (if updating internal packages)Breaking
label has been applied or is not necessary