rotationalio / honu

Adaptive consistency replication with reinforcement learning for large scale globally distributed storage.
BSD 3-Clause "New" or "Revised" License
12 stars 2 forks source link

namespaces with double colons bug. #14

Closed DanielSollis closed 2 years ago

DanielSollis commented 2 years ago

Fix the issue where LevelDBEngine.Iter currently grabs the namespace by splitting on double colons (::). In the case of a namespace called for example "namespace::with::colons", Iter considers the namespace to be "namespace" and will not access the database correctly. As for a possible solution, with PR #12 the namespace is stored in the iterator, meaning we would no longer need to split on double colons. The following tests are currently skipping double colon test cases and need to be amended:

bbengfort commented 2 years ago

Fixed by #21