Artem Krylysov

@ortuman did you manage to solve this issue?

Experiencing the same issue after a failover, +1 for treating `BusyLoadingError` the same way as `MovedError`.

Thanks for the PR! Do you mind adding some tests? Take a look at https://github.com/akrylysov/algnhsa/blob/master/adapter_test.go

Thanks for the contribution. - Could you please give more details on why it's needed? - Tests are failing.

Unfortunately just changing the constant to `math.MaxUint64` won't work. Pogreb uses a 32-bit hash function. Storing more than `math.MaxUint32` keys without changing the hash function to a 64-bit version would...

Thanks for the details! While the database will get slower as it gets close to 4 billion keys, it won't impact correctness, you don't need to worry about false positives....

Right now Pogreb doesn't support opening same database from multiple processes. When you open a database, Pogreb creates a lock file. You'll get `database is locked` error if you try...

Fault tolerance / HA is outside of the scope of this project. I don't have plans to add replication at the moment. I recommend taking a look at https://github.com/rqlite/rqlite, it...

Thanks for the bug report. > Unrelated to this problem, but in truncate used by recoveryIterator.next it uses uint32. That could lead to problems down the road for large segment...

> It is better to add checsum for key+value size header to early detect such corruption. Read/write performance and space amplification will take a hit - adding another checksum will...