[kudu-CR] Refactor HybridClock and add a local clock mode

2017-08-28 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: I ended up pulling some of this idea into de8b92eb6469f843d43c432678e52732a861014b though it

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5982/1/src/kudu/server/hybrid_clock.cc File

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5982/1/src/kudu/server/hybrid_clock.cc File

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: (1 comment) todd: just for internal tests and hidden + unsafe, yeah. mj: my problem with

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: If it's possible to avoid the extra flag, that would be awesome, it makes it harder to

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5982/1/src/kudu/server/hybrid_clock.cc File

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5982/1/src/kudu/server/hybrid_clock.cc

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5982/1/src/kudu/common/timestamp.cc File

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5982/1/src/kudu/server/hybrid_clock.cc File

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: (3 comments) thanks! http://gerrit.cloudera.org:8080/#/c/5982/1/src/kudu/common/timestamp.cc

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Refactor HybridClock and add a local clock mode .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5982/1/src/kudu/server/hybrid_clock.cc

[kudu-CR] Refactor HybridClock and add a local clock mode

2017-02-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/5982 Change subject: Refactor HybridClock and add a local clock mode .. Refactor HybridClock and add a local clock mode Some test