Philip Zeyliger has uploaded this change for review. ( 
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
---
M docker/entrypoint.sh
1 file changed, 4 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/37/12137/1
--
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: newchange
Gerrit-Change-Id: I5dd7d823189e00edae4249d436bedfe4dd05a3a1
Gerrit-Change-Number: 12137
Gerrit-PatchSet: 1
Gerrit-Owner: Philip Zeyliger <[email protected]>

Reply via email to