Skip to content
This repository has been archived by the owner on Dec 7, 2019. It is now read-only.

[WIP] File system erase ability #322

Open
wants to merge 1 commit into
base: feature/rx2
Choose a base branch
from

Conversation

bishiboosh
Copy link

I've added the clear functionality to the default file system persisters.

As the clear operation can take some times, I've also changed clear signature to be Completable

return Completable.concat(memCache
.asMap()
.keySet()
.stream()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this java8 stream api? Won't work on android.

@digitalbuddha
Copy link
Contributor

Will hold off to merging until next major store version which we are starting to prepare. This is due to breaking change with current functionality.

@bishiboosh
Copy link
Author

@digitalbuddha sure. I've detected some problems with my code too, so I'll wait until the next version and come back to clean this then

@bishiboosh bishiboosh changed the title File system erase ability [WIP] File system erase ability Mar 30, 2018
@digitalbuddha
Copy link
Contributor

howdy ho, will this be updated or should I close?

@bishiboosh
Copy link
Author

@digitalbuddha I don't think I have the time available right now, close it and I'll take another stab at it later :)

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

Successfully merging this pull request may close these issues.

3 participants