Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

Adding cache entries should set time consistently. #1013

Closed
geauxvirtual opened this issue Jun 21, 2016 · 0 comments
Closed

Adding cache entries should set time consistently. #1013

geauxvirtual opened this issue Jun 21, 2016 · 0 comments
Labels
reviewed/in-progress tracked type/bug

Comments

@geauxvirtual
Copy link
Contributor

There is a current mix use of chrono.Chrono.Now() and time.Now() for setting time when an entry is added to the cache.

https://github.com/intelsdi-x/snap/blob/master/control/strategy/cache.go#L101-L118
kindermoumoute added a commit to kindermoumoute/snap that referenced this issue Jun 28, 2016
@kindermoumoute
Fixed Github intelsdi-x#1013
6967c62 
Standardisation of chrono.Chrono.Now()
@kindermoumoute kindermoumoute mentioned this issue Jun 28, 2016
Merged
kindermoumoute added a commit to kindermoumoute/snap that referenced this issue Jun 28, 2016
@kindermoumoute
Fixes intelsdi-x#1013
7f6918d 
Standardisation of chrono.Chrono.Now() for setting time when an entry is added to the cache.
pittma pushed a commit that referenced this issue Jun 29, 2016
@kindermoumoute @pittma
Fixes #1013 (#1036)
b292715 
Standardisation of chrono.Chrono.Now() for setting time when an entry is added to the cache.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
reviewed/in-progress tracked type/bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bjray @lynxbat @geauxvirtual