Andrea V

Results 17 comments of Andrea V

A very cool nice-to-have. But lower prio for now.

@Wollac is the man here.

For now we could just treat this as a black box that simply returns a value, we do not need to care or define exactly how it works.

I think this is not viable due to the huge amount of IO that the DB can receive at certain stages of node startup or operations. The network and the...

I also still have the same problem, what's the issue in fixing it?

Hey @ronzulu, first of all: thanks for providing feedback on the issue =D. I was taking the matter into my own hands and started debugging the issue: the problem seems...

Hi again @ronzulu I've just opened a PR that seems to fix the race condition: indeed the best solution seems to be to directly update the due information within the...

Hey @ronzulu: I've refactored the sync so that a new method updates and sort the due notes using the pre-populated review decks. This method is the one called by the...

Your change looks good now :wink: Regarding the race fix: should we opt for the one-off handler or the PR as-is? As you have way more experience with the project...

Added to the changelog that #821 fixes #424 and #582 . Let's wait for @axb21 or proceed anyway in a week's time?