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 <t...@sss.pgh.pa.us>
        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 <t...@sss.pgh.pa.us>
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 <t...@sss.pgh.pa.us>
        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

----


---

Reply via email to