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.

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

Fixed timestamp zone format #2497

Merged
merged 2 commits into from
Aug 22, 2024
Merged

Fixed timestamp zone format #2497

merged 2 commits into from
Aug 22, 2024

Conversation

tkyc
Copy link
Member

@tkyc tkyc commented Aug 22, 2024

Fixes #2490

Jeffery-Wasty
Jeffery-Wasty previously approved these changes Aug 22, 2024
Copy link

codecov bot commented Aug 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.66%. Comparing base (c05521e) to head (457f27d).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2497      +/-   ##
============================================
+ Coverage     50.59%   50.66%   +0.07%     
- Complexity     3814     3823       +9     
============================================
  Files           145      145              
  Lines         33135    33133       -2     
  Branches       5559     5558       -1     
============================================
+ Hits          16763    16786      +23     
+ Misses        13972    13943      -29     
- Partials       2400     2404       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

barryw-mssql
barryw-mssql previously approved these changes Aug 22, 2024
@tkyc tkyc dismissed stale reviews from barryw-mssql and Jeffery-Wasty via dec9d30 August 22, 2024 20:46
@tkyc tkyc merged commit fe56e25 into main Aug 22, 2024
2 of 16 checks passed
@tkyc tkyc deleted the timestamp-zone-fix branch August 22, 2024 20:48
tkyc added a commit that referenced this pull request Aug 22, 2024
* Fixed timestamp zone format

* Fixed test, closed RS
tkyc added a commit that referenced this pull request Aug 22, 2024
* Fixed timestamp zone format

* Fixed test, closed RS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

12.6.3+ changed behavior on how it handles timezones for jdbcType TIMESTAMP
4 participants