Compact doesn't work with synced realms

[nirinchev]

When I open a synchronized Realm and call compact on it, I get a bad history type exception. After brief debugging with the help of @fealebenpae, we concluded that do_open is called at the end of compact, then in the if (Replication* repl = m_group.get_replication()), m_group is empty, so repl is always nullptr and the history_type is never set to the correct value.

[finnschiermer]

It turns out to be a more complicated problem. Compact doesn't write out the history part of the Realm file. Apparently this confuses the later attempt to reopen the file as a synchronized realm.

[hsnetzer]

shouldCompactOnLaunch seems to work as a config option on my synced Realm on Realm Cloud.

[ironage]

@hsnetzer you're right, this issue was fixed by #3039 so it has been working for some time. We just forgot to close this issue, thanks for the reminder!