Skip to content
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

Validate schema changes made from other processes #2877

Merged
merged 6 commits into from
Nov 19, 2015

Conversation

tgoyne
Copy link
Member

@tgoyne tgoyne commented Nov 18, 2015

When multiple processes are sharing a single Realm file, they can potentially have different schema definitions that may or may not be compatible. This makes differences which do not require a schema version bump (i.e. one process having more or fewer tables than another process or disagreements about whether a property is indexed) explicitly supported, and throws an exception when incompatible schemas are encountered rather than crashing or silently corrupting data.

@@ -27,7 +27,110 @@
using namespace realm;

namespace {
class TransactLogHandler {
// A transaction log handler that just validates that all operations made are
// onces supported by the object store
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/onces/ones/

@bdash
Copy link
Contributor

bdash commented Nov 19, 2015

👍

tgoyne added a commit that referenced this pull request Nov 19, 2015
Validate schema changes made from other processes
@tgoyne tgoyne merged commit 8ceae74 into master Nov 19, 2015
@tgoyne tgoyne removed the Review label Nov 19, 2015
@tgoyne tgoyne deleted the tg-allowed-schema-changes branch November 19, 2015 18:39
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants