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

UI Freeze when using RushCore.getInstance().deleteAll(Class) #111

Closed
zoharyosef opened this Issue Jun 26, 2016 · 3 comments

Comments

Projects
None yet
2 participants
@zoharyosef

zoharyosef commented Jun 26, 2016

Only happens after a significant amount of re-installs.
On LogCat the following line appears on every freeze:
D/RushOrm: VACUUM;

It gets worse and worse, I got to 40 sec freeze, and only after i erased the app and re-installed, it was gone, temporarily.

As a workaround now i'm using delete() after RushSearch(), and there are no freezes, for now...

Thanks for this great lib! and Good luck

@Stuart-campbell

This comment has been minimized.

Show comment
Hide comment
@Stuart-campbell

Stuart-campbell Jun 26, 2016

Owner

Opps can't believe I forgot the async version of that call. I will have to get that added.

Thanks

Owner

Stuart-campbell commented Jun 26, 2016

Opps can't believe I forgot the async version of that call. I will have to get that added.

Thanks

Stuart-campbell pushed a commit that referenced this issue Jun 26, 2016

Stuart Campbell Stuart Campbell
Added deleteAll async method to resolve issue #111. Also now require …
…a new meta data tag to define the package that contains the Rush Classes to resolve the issues with other libraries.
@Stuart-campbell

This comment has been minimized.

Show comment
Hide comment
@Stuart-campbell

Stuart-campbell Jun 26, 2016

Owner

As of v1.2.0 you can not pass in a RushClassback into the deleteAll function which makes it async.

Thanks

Owner

Stuart-campbell commented Jun 26, 2016

As of v1.2.0 you can not pass in a RushClassback into the deleteAll function which makes it async.

Thanks

@zoharyosef

This comment has been minimized.

Show comment
Hide comment
@zoharyosef

zoharyosef commented Jun 28, 2016

Thanks!

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