Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update ttl for updated nodes only #2740

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

butonic
Copy link
Contributor

@butonic butonic commented Nov 15, 2024

Followup to #2736

obviously, you were right: edge cases are hard to do properly 😞

Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@asim asim merged commit e032a6a into micro:master Nov 15, 2024
1 check failed
@asim
Copy link
Member

asim commented Nov 15, 2024

Followup to #2736

obviously, you were right: edge cases are hard to do properly 😞

Yea no worries it takes some attempts and seeing it in action. Eventually I found simplicity beat out trying to do clever things because the complexity of maintenance became a burden and I was not in the environments it was being used. If you have first hand exposure to the usage then you can tweak it in the real world and everyone else will benefit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants