[Impala-ASF-CR] IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s

2016-10-04 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s .. Patch Set 2: Code-Review+2 Carry +2. -- To view, visit

[Impala-ASF-CR] IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s

2016-10-04 Thread Sailesh Mukil (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4622 to look at the new patch set (#2). Change subject: IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s

[Impala-ASF-CR] IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s

2016-10-04 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s .. Patch Set 1: Code-Review+2 (2 comments) Carry +2.

[Impala-ASF-CR] IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s

2016-10-04 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4246: SleepForMs() utility function has undefined behavior for > 1s .. Patch Set 1: Code-Review+2 (2 comments)