Impala Public Jenkins has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/12137 )
Change subject: Fix centos/Java/ORC default timezones in test-with-docker.
......................................................................
Fix centos/Java/ORC default timezones in test-with-docker.
Stops configuring /etc/timezone for CentOS machines, which don't
typically have this file. Uses a longer format ("tz database name") for
/etc/timezone for Ubuntu, since that's what Ubuntu seems to expect. The
existing approach seemed to work, but it seems more consistent to use
the tz name.
To debug this, I wrote the following Java program:
import java.util.TimeZone;
public class test {
public static void main(String[] args) {
System.out.println(TimeZone.getDefault());
}
}
Running it under strace, with the OpenJDK source open to
src/solaris/native/java/util/TimeZone_md.c, I was able to
spot the issue. My previous attempt (IMPALA-7698,
c1701074d6e94d98a43ab049ef807ac1b368180f)
tread down this same path, but I had missed the failure.
Change-Id: I5dd7d823189e00edae4249d436bedfe4dd05a3a1
Reviewed-on: http://gerrit.cloudera.org:8080/12137
Reviewed-by: Impala Public Jenkins <[email protected]>
Reviewed-by: Laszlo Gaal <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M docker/entrypoint.sh
1 file changed, 4 insertions(+), 1 deletion(-)
Approvals:
Impala Public Jenkins: Looks good to me, approved; Verified
Laszlo Gaal: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.cloudera.org:8080/12137
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I5dd7d823189e00edae4249d436bedfe4dd05a3a1
Gerrit-Change-Number: 12137
Gerrit-PatchSet: 3
Gerrit-Owner: Philip Zeyliger <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
