Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: binary timestamptz -> getString should add +XX zone offset to text representation #490

Merged
merged 1 commit into from
Mar 26, 2016
Merged

fix: binary timestamptz -> getString should add +XX zone offset to text representation #490

merged 1 commit into from
Mar 26, 2016

Conversation

vlsi
Copy link
Member

@vlsi vlsi commented Jan 12, 2016

Note: tstz in binary is sent without timezone, thus ParameterStatus(TimeZone = GMT-01:00) should be used to track connection's time zone

closes #130

@vlsi vlsi mentioned this pull request Jan 12, 2016
Closed
@vlsi vlsi force-pushed the tstz_getstring branch 2 times, most recently from f5a3bcd to 7b64893 Compare March 22, 2016 09:01
@vlsi
fix: binary timestamptz -> getString should add +XX zone offset to te…
1e1f3c4 
…xt representation

Note: tstz in binary is sent without timezone, thus arameterStatus(TimeZone = GMT-01:00) is used to track connection's time zone

closes #130
@codecov-io
Copy link

Current coverage is 56.23%

Merging #490 into master will decrease coverage by -1.22% as of 507b027

@@            master    #490   diff @@
======================================
  Files          143     143       
  Stmts        15129   15166    +37
  Branches      2971    2979     +8
  Methods          0       0       
======================================
- Hit           8693    8529   -164
+ Partial       1180    1137    -43
- Missed        5256    5500   +244

Review entire Coverage Diff as of 507b027

Powered by Codecov. Updated on successful CI builds.

@vlsi vlsi merged commit 1e1f3c4 into master Mar 26, 2016
@vlsi vlsi deleted the tstz_getstring branch March 26, 2016 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Prepared statement pool and Daylight Saving Time
2 participants
@vlsi @codecov-io