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

ttljob: fix a range edge case #81207

Merged
merged 2 commits into from
May 14, 2022
Merged

Conversation

otan
Copy link
Contributor

@otan otan commented May 12, 2022

See individual commits for details.

Refs: #81208

Release note: None
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@otan otan requested a review from rafiss May 12, 2022 12:37
@otan otan marked this pull request as ready for review May 12, 2022 12:38
@otan otan requested a review from a team May 12, 2022 12:38
@otan otan force-pushed the fix_stuff branch 6 times, most recently from 263163b to bfd1754 Compare May 13, 2022 20:02
Release note (bug fix): Fix a bug in row-level TTL where the last range
key of a table may overlap with a separate table or index, resulting in
a "error decoding X bytes" error message when performing row-level TTL.
Copy link
Collaborator

@rafiss rafiss left a comment

Choose a reason for hiding this comment

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

lgtm.. but there probably are other folks out there who know range encoding better than me, so idk if there's a different way

@otan
Copy link
Contributor Author

otan commented May 14, 2022

i looked at how SHOW RANGES FOR TABLE worked and uh yeah this is what i found :D!

bors r=rafiss

@craig
Copy link
Contributor

craig bot commented May 14, 2022

Build succeeded:

@craig craig bot merged commit 16c484a into cockroachdb:master May 14, 2022
@blathers-crl
Copy link

blathers-crl bot commented May 14, 2022

Encountered an error creating backports. Some common things that can go wrong:

  1. The backport branch might have already existed.
  2. There was a merge conflict.
  3. The backport branch contained merge commits.

You might need to create your backport manually using the backport tool.


error creating merge commit from 036f3f5 to blathers/backport-release-22.1.0-81207: POST https://api.github.com/repos/cockroachdb/cockroach/merges: 409 Merge conflict []

you may need to manually resolve merge conflicts with the backport tool.

Backport to branch 22.1.0 failed. See errors above.


🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is otan.

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