-
Notifications
You must be signed in to change notification settings - Fork 479
Removed known limitations #21472
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
Removed known limitations #21472
Conversation
Removed known limitations that were fixed in 25.4.1
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
Files changed:
|
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
| @@ -1,2 +0,0 @@ | |||
| - The `ltree2text` function produces incorrect results by wrapping the output in single quotes. For example, `ltree2text('foo.bar.baz'::LTREE)` returns `'foo.bar.baz'` instead of `foo.bar.baz`. [#156479](https://github.com/cockroachdb/cockroach/issues/156479) | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it worth deleting this file entirely, since it's no longer going to be used? I guess it's possible more limitations will be added in the future, but IDK about keeping it around for that possibility.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd be happy to delete it, I'm all for cleaning things up
Deleted empty file entirely rather than keep it around just in case
taroface
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, although this change makes me realize we should probably cite the latest patch version on this page (i.e. 25.4.1) rather than the major version (25.4), since the KL technically still exists on 25.4.0. I'll create an issue for that.
Fixes: DOC-15303 and DOC-15323
Removed known limitations with LTREE that were fixed in 25.4.1. Fix PRs: #156779 and #156667