Skip to content
This repository has been archived by the owner on Sep 21, 2024. It is now read-only.

dev: Expose replication/publication/ttl intervals to NameSystemBuilder #130

Merged
merged 1 commit into from
Nov 9, 2022
Merged

dev: Expose replication/publication/ttl intervals to NameSystemBuilder #130

merged 1 commit into from
Nov 9, 2022

Conversation

jsantell
Copy link
Contributor

@jsantell jsantell commented Nov 8, 2022

Fixed #127

@jsantell jsantell requested a review from cdata November 8, 2022 21:06
cdata
cdata approved these changes Nov 9, 2022
View reviewed changes
Copy link
Collaborator

@cdata cdata left a comment

Choose a reason for hiding this comment

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

This looks good. I am assuming that these values weigh mostly on the behavior of the DHT, and are somewhat orthogonal to the time-based validity of UCANs in record values. Is that right?

@jsantell
Copy link
Contributor Author

jsantell commented Nov 9, 2022

This looks good. I am assuming that these values weigh mostly on the behavior of the DHT, and are somewhat orthogonal to the time-based validity of UCANs in record values. Is that right?

Aye, unrelated to UCAN tokens' time-bounds

@cdata cdata merged commit e20680e into main Nov 9, 2022
@cdata cdata deleted the dhtreplication branch November 9, 2022 19:17
@github-actions github-actions bot mentioned this pull request Nov 9, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cdata cdata cdata approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

NameSystem must propagate records within configured interval
2 participants
@jsantell @cdata