[bisq-network/bisq] Reduce initial request size (#4233)
Florian Reimair
notifications at github.com
Tue Jun 16 12:31:01 UTC 2020
@freimair commented on this pull request.
> + log.error("Could not copy resourceFile {} to {}.\n{}", resourceFileName, destinationFile.getAbsolutePath(), e.getMessage());
+ e.printStackTrace();
+ }
+
+ // split
+ // - get all
+ SplitStore historicalStore = readStore(destinationFile.getName());
+ // - subtract all that is in resource files
+ store.getMap().keySet().removeAll(historicalStore.getMap().keySet());
+
+ // - create new file with leftovers
+ storage.initAndGetPersisted(store, 0);
+ storage.queueUpForSave();
+
+ return historicalStore;
+ }
I see this whole PR a small-as-can-be bugfix for the 3/4 problem. The "followup"-project would be to remove the protobuf-files altogether and make everything work with a proper database (https://github.com/bisq-network/projects/issues/29). When we approach that project, all the file-handling will be removed and cleaned up.
I agree that the implementation is not as clean as could be, but I rather focus my efforts and not do stuff twice.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4233#discussion_r440811638
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200616/51ae56f1/attachment.html>
More information about the bisq-github
mailing list