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

Sync on shut down #260

Closed
LakshmiCHAI opened this issue Oct 8, 2015 · 5 comments
Closed

Sync on shut down #260

LakshmiCHAI opened this issue Oct 8, 2015 · 5 comments

Comments

@LakshmiCHAI
Copy link

Initiate a sync when the device is being shut down

@garymabin
Copy link
Contributor

Syncing is a time-consuming job, even we can known the time the device is
about to shut down, we can not ensure the sync is finished. Actually the
power-off process will kill all the process that is running.

Bin MaConsultantEmailbinma@thoughtworks.comTelephone+86 13125033274
<+86+13125033274>[image: ThoughtWorks]
http://www.thoughtworks.com/?utm_campaign=bin-ma-signature&utm_medium=email&utm_source=thoughtworks-email-signature-generator

2015-10-09 3:27 GMT+08:00 LakshmiCHAI notifications@github.com:

Initiate a sync when the device is being shut down


Reply to this email directly or view it on GitHub
#260.

@chai-lmis
Copy link

I don't think this is a good idea, as the sync may be interrupted if the shutdown is forced and this may cause the app to crash. I already noticed that when the sync on demand was unsuccessful it caused the app to crash.
Egwuma

@chai-lmis
Copy link

i believe we can close this card as its not a viable option?

@ihassin
Copy link
Contributor

ihassin commented Oct 9, 2015

Please do

Sent using CloudMagic Email
[https://cloudmagic.com/k/d/mailapp?ct=ti&cv=7.4.8&pv=9.0.2&source=email_footer_2]
On Fri, Oct 9, 2015 at 04:27, clintonhealthaccess/chailmis-android
reply@reply.github.com
wrote:
i believe we can close this card as its not a viable option?


Reply to this email directly or view it on GitHub
[https://github.com//issues/260#issuecomment-146795731] .[https://github.com/notifications/beacon/AABKPgzuYaKWvvmLsQ5ZBKDWNC0wS1_nks5o53HcgaJpZM4GLm4A.gif]

@karloskalcium
Copy link
Contributor

Closed per Bin Ma, since we can't ensure this completes before the device shuts down.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants