Skip to content

roachpb: remove txn.DeprecatedOrigTimestamp#51440

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
andreimatei:delete-deprecated_orig_timestamp
Jul 16, 2020
Merged

roachpb: remove txn.DeprecatedOrigTimestamp#51440
craig[bot] merged 1 commit intocockroachdb:masterfrom
andreimatei:delete-deprecated_orig_timestamp

Conversation

@andreimatei
Copy link
Copy Markdown
Contributor

This field was left over in 20.1 only for compatibility with 19.2.
History is in #42236 - the PR that deprecated the field.

Release note: None

@andreimatei andreimatei requested a review from nvb July 14, 2020 22:13
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Contributor

@nvb nvb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm: 👋

Reviewed 16 of 16 files at r1.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained

This field was left over in 20.1 only for compatibility with 19.2.
History is in cockroachdb#42236 - the PR that deprecated the field.

Release note: None
@andreimatei andreimatei force-pushed the delete-deprecated_orig_timestamp branch from 482c67b to bd566f8 Compare July 14, 2020 23:41
Copy link
Copy Markdown
Contributor Author

@andreimatei andreimatei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (and 1 stale) (waiting on @nvanbenschoten)

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Jul 16, 2020

Build succeeded

@craig craig bot merged commit dc736e8 into cockroachdb:master Jul 16, 2020
@andreimatei andreimatei deleted the delete-deprecated_orig_timestamp branch July 22, 2020 15:02
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.

3 participants