-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Multiprocess support #1300
Multiprocess support #1300
Conversation
1eda8fe
to
64c1390
Compare
The new notification based on wait_for_change won't guarantee how many times the notification will be sent.
ae7011e
to
0757c5c
Compare
Rename to RealmChangeDaemon and java doc added.
Right now, merging this PR means more virtual spaces are needed for the shared group in RealmDaemon. Need realm/realm-core#838 to be implemented first. |
import io.realm.internal.log.RealmLog; | ||
|
||
/** | ||
* Daemon thread sleeps until one ore more commits have been made to the corresponding Realm. When change happens, the |
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.
s/ore/or
Native Wait for change method is notified by changes in other process? 😮 |
I guess the question is when multiprocess will be supported for migrations / schema changes / lock version change. |
@Zhuinden In theory with the things we have merged to master, interprocess should be fully supported now, but we still need to review it with that in mind. At least last we checked there was a few unfortunate race conditions when opening Realms, e.g opening a Realm for the first time in a process requires a write transaction and if another process is writing at the same time, you suddenly completely blocked one of the processes. Which is why we are not officially saying it is "done" yet 😄 |
@cmelchior Is there any news on this issue? I still need multiprocess support :/ |
Hello, Is multi process now supported in realm 3 ?. I also have a question here about multi-process: http://stackoverflow.com/questions/42733931/realm-multi-process-best-practice |
This PR is like, super-outdated 😄 there is no such thing as |
Replaced by #5473 |
This PR will test the possibility of current status of multi process first. Then maybe add more features to make multi process work better.
Fixes #1091
BaseRealm
(Optional)