diff --git a/common/consistenthash/consistenthash.go b/common/consistenthash/consistenthash.go index dde99a9..dfd71fc 100644 --- a/common/consistenthash/consistenthash.go +++ b/common/consistenthash/consistenthash.go @@ -19,7 +19,13 @@ type Ring struct { // includes the hashed value of key. func (r *Ring) search(key string) int { ///////////////////////// - // YOUR CODE GOES HERE // + i := 0 + for _, node := range r.Nodes { + if hashId(node.Id) >= hashId(key) { + return i + } + i++ + } ///////////////////////// return 0 diff --git a/dynamo/cache.go b/dynamo/cache.go index 0483a39..34dce3f 100644 --- a/dynamo/cache.go +++ b/dynamo/cache.go @@ -12,6 +12,7 @@ import ( // Cache is the struct that handle all the data storage for the dynamo server. type Cache struct { data map[string]string + timestamp map[string]int64 sync.Mutex } @@ -20,6 +21,7 @@ func NewCache() *Cache { var s Cache s.data = make(map[string]string) + s.timestamp = make(map[string]int64) return &s } @@ -29,7 +31,7 @@ func NewCache() *Cache { func (cache *Cache) Get(key string) (value string, timestamp int64) { cache.Lock() value = cache.data[key] - timestamp = 0 + timestamp = cache.timestamp[key] cache.Unlock() log.Printf("[CACHE] Getting Key '%v' with Value '%v' @ timestamp '%v'\n", key, value, timestamp) @@ -43,6 +45,7 @@ func (cache *Cache) Put(key string, value string, timestamp int64) { cache.Lock() cache.data[key] = value + cache.timestamp[key] = timestamp cache.Unlock() return @@ -52,6 +55,6 @@ func (cache *Cache) Put(key string, value string, timestamp int64) { // except for testing purposes. func (cache *Cache) getAll() (data map[string]string, timestamps map[string]int64) { data = cache.data - timestamps = make(map[string]int64) + timestamps = cache.timestamp return data, timestamps } diff --git a/dynamo/coordinator.go b/dynamo/coordinator.go index 408cc9e..52fc678 100644 --- a/dynamo/coordinator.go +++ b/dynamo/coordinator.go @@ -191,8 +191,18 @@ func aggregateVotes(votes []*vote) (result string) { } ///////////////////////// - // YOUR CODE GOES HERE // + var ( + timest int64 + i int + ) + timest = votes[0].timestamp + for n, vote := range votes { + if timest < vote.timestamp { + timest = vote.timestamp + i = n + } + } ///////////////////////// - result = votes[0].value + result = votes[i].value return }