-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
connect to PGSQL 10.6. but error with Timezone class #19
Comments
Hmm, ROK seems to be a deprecated abbreviation for Asia/Seoul. ZTimezone does understand all modern timezone designations (the long ones in /etc/share/zonetab) but only a few abbreviations. Could you declare/configure your database to use Asia/Seoul as timezone ? The problem is that P3Client tries to automatically adapt to the timezone used by the database. You can have a look at P3ClientTest>>#testTimezone where there are some experiments with changing the connection's time zone. Maybe I should extend https://github.com/svenvc/ztimestamp to undestand all abbreviations that occur in /etc/share/zonetab - I have to think about that. |
I guess. Even if I set it to "Asia/Seoul" on linux system, "ROK" is returned. Is there a way to tell if this is a problem with ZTimestamp or a feature of the linux system? |
Hi, I extended ZTimestamp's ZTimezone to support less common IDs like ROK (in addition to the modern Asia/Seoul variant). Could you try by loading the latest master version from https://github.com/svenvc/ztimestamp please ? BTW, this might be helpful too https://phili.pe/posts/timestamps-and-time-zones-in-postgresql/ It might also be useful to give P3Client an option to better manage P3Client>>#initializeConverter and P3Converter>>#initializeFrom: when unknown values are encountered. I will look into that too. Sven |
I will close this issue soon as fixed, as it should work from my POV. |
See #testROK |
server : postgresql 10.6 with gentoo linux
pharo : pharo 7.0
test code :
but encounterd this error...
id Value is "ROK". how to solve this?
The text was updated successfully, but these errors were encountered: