- Notifications
You must be signed in to change notification settings - Fork4.9k
Commitaac0384
committed
Avoid using timezone Asia/Manila in regression tests.
The freshly-released 2025a version of tzdata has a refined estimatefor the longitude of Manila, changing their value for LMT inpre-standardized-timezone days. This changes the output of one ofour test cases. Since we need to be able to run with system tzdatafiles that may or may not contain this update, we'd better stopmaking that specific test.I switched it to use Asia/Singapore, which has a roughly similar UTCoffset. That LMT value hasn't changed in tzdb since 2003, so we canhope that it's well established.I also noticed that this set of make_timestamptz tests only exerciseszones east of Greenwich, which seems rather sad, and was not theoriginal intent AFAICS. (We've already changed these tests onceto stabilize their results across tzdata updates, cf66b737c;it looks like I failed to consider the UTC-offset-sign aspect then.)To improve that, add a test with Pacific/Honolulu. That LMT offsetis also quite old in tzdb, so we'll cross our fingers that it doesn'tget improved.Reported-by: Christoph Berg <cb@df7cb.de>Discussion:https://postgr.es/m/Z46inkznCxesvDEb@msg.df7cb.deBackpatch-through: 131 parenta2d4f80 commitaac0384
File tree
3 files changed
+11
-4
lines changed- src
- include/datatype
- test/regress
- expected
- sql
3 files changed
+11
-4
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
96 | 96 |
| |
97 | 97 |
| |
98 | 98 |
| |
99 |
| - | |
| 99 | + | |
100 | 100 |
| |
101 | 101 |
| |
102 | 102 |
| |
|
Lines changed: 8 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2277 | 2277 |
| |
2278 | 2278 |
| |
2279 | 2279 |
| |
2280 |
| - | |
| 2280 | + | |
2281 | 2281 |
| |
2282 | 2282 |
| |
2283 |
| - | |
| 2283 | + | |
| 2284 | + | |
| 2285 | + | |
| 2286 | + | |
| 2287 | + | |
| 2288 | + | |
| 2289 | + | |
2284 | 2290 |
| |
2285 | 2291 |
| |
2286 | 2292 |
| |
|
Lines changed: 2 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
407 | 407 |
| |
408 | 408 |
| |
409 | 409 |
| |
410 |
| - | |
| 410 | + | |
| 411 | + | |
411 | 412 |
| |
412 | 413 |
| |
413 | 414 |
| |
|
0 commit comments
Comments
(0)