Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

Use separate namespace for IPNS persistent cache #1613

Merged
merged 3 commits into from
Jun 13, 2019
Merged

Conversation

cpacia
Copy link
Member

@cpacia cpacia commented May 29, 2019

Our IPNS package implements a separate persistent cache for IPNS records. The
format of the value stored in the database is different from the format used
by the DHT package. This commit changes the database key used by the persistent
cache to make sure it does not collide with the key used by the DHT package
which might otherwise cause records to be overriden with a different format.

closes #1612

cpacia added 3 commits May 1, 2019 10:32
Best effort delete ipns record if unmarshaling fails
Our IPNS package implements a separate persistent cache for IPNS records. The
format of the value stored in the database is different from the format used
by the DHT package. This commit changes the database key used by the persistent
cache to make sure it does not collide with the key used by the DHT package
which might otherwise cause records to be overriden with a different format.

closes #1612
@cpacia cpacia requested a review from placer14 May 29, 2019 19:49
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 35.618% when pulling abb6795 on ipnscache into cacb790 on master.

Copy link
Member

@placer14 placer14 left a comment

Choose a reason for hiding this comment

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

This seems okay to me. I'm not sure if we want to merge this until we understand how it impacts users. Would like to get some testing around this first.

@placer14 placer14 changed the base branch from master to v0.13.3-hotfix June 10, 2019 13:31
@placer14 placer14 changed the base branch from v0.13.3-hotfix to v0.13.4-rc June 13, 2019 19:17
Copy link
Member

@placer14 placer14 left a comment

Choose a reason for hiding this comment

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

After some testing, this seems to work pretty well. Preparing this for the v0.13.4 release.

@placer14 placer14 merged commit e64d3bb into v0.13.4-rc Jun 13, 2019
@placer14 placer14 deleted the ipnscache branch June 13, 2019 19:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants