Re: Review Request 45721: thermos local-time

2016-04-21 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45721/#review130023 --- Se Choi, I wanted to wake this thread and see if you intended to

Re: Review Request 45721: thermos local-time

2016-04-05 Thread Joshua Cohen
> On April 5, 2016, 2:17 a.m., John Sirois wrote: > > Do you have any thoughts on why localtime should be the winner here? > > > > Imagine these scenarios: > > You're an operator for a cluster that lives 3 timezones away > > You're an operator for 3 clusters, each in different timezones. > > >

Re: Review Request 45721: thermos local-time

2016-04-05 Thread Stephan Erb
> On April 5, 2016, 4:17 a.m., John Sirois wrote: > > Do you have any thoughts on why localtime should be the winner here? > > > > Imagine these scenarios: > > You're an operator for a cluster that lives 3 timezones away > > You're an operator for 3 clusters, each in different timezones. > > >

Re: Review Request 45721: thermos local-time

2016-04-04 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45721/#review127008 --- Do you have any thoughts on why localtime should be the winner

Re: Review Request 45721: thermos local-time

2016-04-04 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45721/#review127003 --- Master (f402899) is green with this patch.

Review Request 45721: thermos local-time

2016-04-04 Thread se choi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45721/ --- Review request for Aurora. Repository: aurora Description --- thermos