-
Notifications
You must be signed in to change notification settings - Fork 722
Extend support of multiple columns in fulltext index #25337
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
Conversation
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat | Test bloat
🟢 |
fb9cb81
to
5c13d3a
Compare
⚪ ⚪ Ya make output | Test bloat | Test bloat
🟢 |
⚪
🟢 |
Changelog entry
...
Changelog category
Description for reviewers
In addition to #24439.
I've realized, that some text columns may be simultaniously used as fulltext and as keys (for example git/arcradia branches).
So, I've propose to transform
FulltextIndexSettings.columns
list into all index key columns, not only fulltext ones.This allows us to make this syntax work (
title
column is indexed in two ways):Also separated columns list validation for better readability.