Skip to content

VReplication: Set time zone to UTC while streaming rows#6845

Merged
rohit-nayak-ps merged 2 commits intovitessio:masterfrom
planetscale:rn-rowstreamer-set-tz-utc
Oct 22, 2020
Merged

VReplication: Set time zone to UTC while streaming rows#6845
rohit-nayak-ps merged 2 commits intovitessio:masterfrom
planetscale:rn-rowstreamer-set-tz-utc

Conversation

@rohit-nayak-ps
Copy link
Copy Markdown
Member

@rohit-nayak-ps rohit-nayak-ps commented Oct 9, 2020

Currently we set the timezone of the mysql connection to be UTC while replicating. However during the copy phase it is still using the default timezone. This causes columns of the timestamp datatype to be incorrect on the target.

Signed-off-by: Rohit Nayak rohit@planetscale.com

@rohit-nayak-ps rohit-nayak-ps changed the title Set time zone to UTC while streaming rows VReplication: Set time zone to UTC while streaming rows Oct 9, 2020
@rohit-nayak-ps rohit-nayak-ps marked this pull request as ready for review October 14, 2020 19:12
@shlomi-noach shlomi-noach requested a review from a team October 15, 2020 06:16
@deepthi
Copy link
Copy Markdown
Collaborator

deepthi commented Oct 22, 2020

@rohit-nayak-ps we should also port this over to release-8.0, correct?

@sougou
Copy link
Copy Markdown
Contributor

sougou commented Oct 22, 2020

Can be merged after conflicts are resolved.

Signed-off-by: Rohit Nayak <rohit@planetscale.com>
…timestamps

Signed-off-by: Rohit Nayak <rohit@planetscale.com>
@rohit-nayak-ps rohit-nayak-ps force-pushed the rn-rowstreamer-set-tz-utc branch from 76a6417 to c15e4b0 Compare October 22, 2020 05:56
@rohit-nayak-ps rohit-nayak-ps merged commit 87bf08a into vitessio:master Oct 22, 2020
@shlomi-noach shlomi-noach deleted the rn-rowstreamer-set-tz-utc branch October 22, 2020 06:53
@deepthi deepthi mentioned this pull request Oct 22, 2020
deepthi added a commit that referenced this pull request Oct 22, 2020
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.

5 participants