Philip Zeyliger has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11731 )

Change subject: IMPALA-7698: Add centos support to bootstrap_system.
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11731/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11731/1//COMMIT_MSG@29
PS1, Line 29: I'm not yet clear on why the ORC library is choosing to open that 
time zone
            : file (which doesn't exist), but this change seems workable even 
with the test
            : failures.
> About the ORC timezone issues:
For this particular change, I'm planning on not doing anything with Orc. I 
think it's useful to add centos6 support. If only data load goes through (which 
it does), we can catch a bunch of py2.6/py2.7 compatibility things during GVO, 
which is extremely handy.

I'm still puzzled about why physical machines that we use with CentOS 6 have 
different results. It might be as simple as non-determinism in file listing 
order or something. I think the next thing to do to figure it out might be 
either (a) debugger or (b) taking the files generated within the Docker context 
and copying them to the hardware context, or vice-versa, but I'm inclined to 
ignore that for now.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id54294d7607f51de87a9de373dcfc4a33f4bedf5
Gerrit-Change-Number: 11731
Gerrit-PatchSet: 1
Gerrit-Owner: Philip Zeyliger <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Philip Zeyliger <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Comment-Date: Fri, 19 Oct 2018 16:43:58 +0000
Gerrit-HasComments: Yes

Reply via email to