go-distributed / meritop

We moved to https://github.com/taskgraph/taskgraph
Apache License 2.0
11 stars 3 forks source link

IncEpoch should use CAS for epoch check #61

Closed hongchaodeng closed 9 years ago

hongchaodeng commented 9 years ago
  1. cleanup panic: use log.Fatal in most cases for clear output in failure.
  2. IncEpoch race: the check doesn't guarantee consistency. It should use CAS. A better solution, though no written here, is to use the version we get from the response.