Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15425 )

Change subject: [KUDU-2632] Replaced new Date(0) with 
DateUtil.epochDaysToSqlDate(0)
......................................................................

[KUDU-2632] Replaced new Date(0) with DateUtil.epochDaysToSqlDate(0)

"new Date" creates different dates depending on timezone. That causes
tests to fail. In order to avoid it, "new Date" is replaced with
DateUtil.epochDaysToSqlDate(0) which internally calls LocalDate
which is timezone irrespective.

Change-Id: Ic381bac8c68499445b6f00a1a76881dac513aede
Reviewed-on: http://gerrit.cloudera.org:8080/15425
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke <[email protected]>
Tested-by: Grant Henke <[email protected]>
---
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKeyEncoding.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestPartialRow.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestRowResult.java
M java/kudu-test-utils/src/main/java/org/apache/kudu/test/ClientTestUtil.java
4 files changed, 12 insertions(+), 8 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Grant Henke: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/15425
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic381bac8c68499445b6f00a1a76881dac513aede
Gerrit-Change-Number: 15425
Gerrit-PatchSet: 2
Gerrit-Owner: Volodymyr Verovkin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to