[kudu-CR] Rename LogicalClock::NowForMetrics() to GetCurrentTime()

2016-11-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Rename LogicalClock::NowForMetrics() to GetCurrentTime() .. Rename LogicalClock::NowForMetrics() to GetCurrentTime() This renames this method and

[kudu-CR] Rename LogicalClock::NowForMetrics() to GetCurrentTime()

2016-11-09 Thread David Ribeiro Alves (Code Review)
Hello Dinesh Bhat, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4989 to look at the new patch set (#4). Change subject: Rename LogicalClock::NowForMetrics() to GetCurrentTime()

[kudu-CR] Rename LogicalClock::NowForMetrics() to GetCurrentTime()

2016-11-08 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: Rename LogicalClock::NowForMetrics() to GetCurrentTime() .. Patch Set 3: > Dinesh: Yeah I saw that and I pondered what you are suggesting. > Ended up not doing

[kudu-CR] Rename LogicalClock::NowForMetrics() to GetCurrentTime()

2016-11-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Rename LogicalClock::NowForMetrics() to GetCurrentTime() .. Patch Set 2: Dinesh: Yeah I saw that and I pondered what you are suggesting. Ended up not

[kudu-CR] Rename LogicalClock::NowForMetrics() to GetCurrentTime()

2016-11-07 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: Rename LogicalClock::NowForMetrics() to GetCurrentTime() .. Patch Set 2: Code-Review+1 Nit: Is there a reason to keep the same naming convention for

[kudu-CR] Rename LogicalClock::NowForMetrics() to GetCurrentTime()

2016-11-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/4989 Change subject: Rename LogicalClock::NowForMetrics() to GetCurrentTime() .. Rename LogicalClock::NowForMetrics() to