-
Notifications
You must be signed in to change notification settings - Fork 168
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
Feature/convert local realm #5136
Conversation
The limit is 100 in debug builds and 1000 in release builds.
Instead of storing change sets in a history, the changes are done directly to a write transaction given as parameter.
@jedelbo Please remember to link to the issue it resolves. |
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.
LGTM, also because this has been reviewed in pieces previously.
src/realm/impl/copy_replication.cpp
Outdated
table->add_column_dictionary(type, name, col_key.is_nullable(), key_type); | ||
} | ||
else { | ||
table->add_column(type, name, col_key.is_nullable()); |
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.
can we add an index to the new column if it exists in the source table?
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.
Sure!
What, How & Why?
"Convert local to synced realm" feature
Fixes #5018
☑️ ToDos