aws-samples / aws-sdk-unity-samples

This repository has samples that demonstrate various aspects of the AWS Mobile SDK for Unity, you can get the SDK source on Github (https://github.com/aws/aws-sdk-net)
Other
160 stars 107 forks source link

Merging in Sync Demo not functioning #8

Closed hgray closed 9 years ago

hgray commented 9 years ago

I've been working on merging in the 2.0 SDK into my code over the last week or so and noticed something weird with my merging functionality. To confirm what I was seeing, I have gone back to the sync demo. I'm no longer getting any merge when I transition from an unauthenticated to an authenticated user.

Here's what I'm doing:

I've checked the Unity console and I'm not getting notified of conflicts or of a merge. It seems that it's just pulling in the authenticated identity but not notifying of the need to merge. In looking at the SQL, I still see my 3 characters from the unauth account, but they don't ever seem to merge in. Obviously clicking "Logout" blows them away.

Any ideas?

karthiksaligrama commented 9 years ago

before you logout, can you confirm the datasets in the SQL locally?

There is currently a bug in the sync manager merge flow where local merge is not triggered immediately in the synchronization process, however they would be triggered in the subsequent synchronization. I suspect that's what is happening here as well.

hgray commented 9 years ago

I've tried a series of flows. I have not been using Logout as I know that wipes the database. I've confirmed all data exists in the SQL, but it's never triggering the merge, even on subsequent logins.

hgray commented 9 years ago

I've also tried repeated "Save" and "Load" clicks to try to force a new merge with no luck.

johnnyhoffman commented 9 years ago

Not able to reproduce when revisiting issue with SDK v 2.1.0.0. Presumably fixed.