Csaba Ringhofer has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11505 )

Change subject: IMPALA-7546: [DOCS] A new TIMEZONE query option
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/11505/1/docs/topics/impala_timezone.xml
File docs/topics/impala_timezone.xml:

http://gerrit.cloudera.org:8080/#/c/11505/1/docs/topics/impala_timezone.xml@45
PS1, Line 45: Impala uses the system time zone where the
            :       client runs.
Impala uses the system time zone where the coordinator impalad runs. I think 
that Hive is different, it uses the client's time zone by default (according to 
HIVE-16614, I have never tested it).

It would make sense to add an option to set the time zone to the client's 
timezone (with this query option this is a quite trivial change in clients), 
but no such thing is supported yet. I would not make this the default, because 
that would be a breaking change.


http://gerrit.cloudera.org:8080/#/c/11505/1/docs/topics/impala_timezone.xml@94
PS1, Line 94: client
See line 46.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I734b8b37ae2360422fce269ed87507a04e8c05ac
Gerrit-Change-Number: 11505
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Attila Jeges <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Tue, 25 Sep 2018 10:15:09 +0000
Gerrit-HasComments: Yes

Reply via email to