GitHub user kuien opened a pull request:
https://github.com/apache/incubator-hawq/pull/1292
HAWQ-1532. recognize timezone 'CST' correctly in China
On some platforms, CST (China Standard Time) is used as time string
suffix of GMT+8, especially for users in China. I tracked that change in
Postgres to this commit:
commit e8dac6f1e09e5bc7604caa6f111d6c80a8b98295
Author: Tom Lane <[email protected]>
Date: Wed Dec 24 16:35:23 2014 -0500
Add CST (China Standard Time) to our lists of timezone
abbreviations.
For some reason this seems to have been missed when the lists in
src/timezone/tznames/ were first constructed. We can't put it in
Default
because of the conflict with US CST, but we should certainly list
it among
the alternative entries in Asia.txt. (I checked for other
oversights, but
all the other abbreviations that are in current use according to
the IANA
files seem to be accounted for.) Noted while responding to bug
#12326.
Then, we may set timezone_abbreviations = 'Asia' to recognize CST
naturally.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kuien/incubator-hawq timezone
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-hawq/pull/1292.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1292
----
commit 4adbb503eca5589f82f583a57b28bf7fc504154d
Author: Tom Lane <[email protected]>
Date: 2014-12-24T21:35:23Z
HAWQ-1532. recognize timezone 'CST' correctly in China
On some platforms, CST (China Standard Time) is used as time string
suffix of GMT+8, especially for users in China. I tracked that change in
Postgres to this commit:
commit e8dac6f1e09e5bc7604caa6f111d6c80a8b98295
Author: Tom Lane <[email protected]>
Date: Wed Dec 24 16:35:23 2014 -0500
Add CST (China Standard Time) to our lists of timezone
abbreviations.
For some reason this seems to have been missed when the lists in
src/timezone/tznames/ were first constructed. We can't put it in
Default
because of the conflict with US CST, but we should certainly list
it among
the alternative entries in Asia.txt. (I checked for other
oversights, but
all the other abbreviations that are in current use according to
the IANA
files seem to be accounted for.) Noted while responding to bug
#12326.
Then, we may set timezone_abbreviations = 'Asia' to recognize CST
naturally.
Change-Id: I386d96f30fc866efcc685a3d5b1bf2b15351c15d
----
---
