Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

Support specified columns for full-text index #1037

Merged

Conversation

bright-starry-sky
Copy link
Contributor

@bright-starry-sky bright-starry-sky commented May 13, 2021

1, Changed full-text index structure, deleted field schema_id
2, Support for creating full-text index against fields
3, Support drop full-text index
4, Support show full-text index
5, Refuse to create full-text index if fixed_string type length > 256
6, Refuse to drop or alter tag if full-text index exists.

depends on

@bright-starry-sky bright-starry-sky added wip Solution: work in progress depend on common PR: this PR depends on PRs in the common repo labels May 13, 2021
@bright-starry-sky bright-starry-sky force-pushed the fulltext_advance branch 2 times, most recently from e749e22 to 020da5c Compare May 17, 2021 12:33
@bright-starry-sky bright-starry-sky requested review from a team and removed request for critical27 May 17, 2021 12:34
@bright-starry-sky bright-starry-sky added depend on storage PR: this PR depends on PRs in the storage repo doc affected Solution: improvements or additions to documentation ready-for-testing PR: ready for the CI test and removed wip Solution: work in progress labels May 17, 2021
@bright-starry-sky
Copy link
Contributor Author

ready to review.

src/util/FTIndexUtils.cpp Outdated Show resolved Hide resolved
src/util/FTIndexUtils.cpp Outdated Show resolved Hide resolved
src/util/FTIndexUtils.cpp Outdated Show resolved Hide resolved
src/parser/parser.yy Outdated Show resolved Hide resolved
@bright-starry-sky
Copy link
Contributor Author

Addressed Yee's comments.

@bright-starry-sky bright-starry-sky merged commit 6808515 into vesoft-inc:master May 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
depend on common PR: this PR depends on PRs in the common repo depend on storage PR: this PR depends on PRs in the storage repo doc affected Solution: improvements or additions to documentation ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants