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

[fix](build index) Forbid building indexes for ngram BF indexes #47041

Merged
merged 6 commits into from
Jan 17, 2025
Prev Previous commit
Next Next commit
[fix](build index) Forbid building indexes for ngram BF indexes
  • Loading branch information
airborne12 committed Jan 15, 2025
commit 42de8ed953d19b95f764c88bfa4bf485c02807e0
Original file line number Diff line number Diff line change
Expand Up @@ -2086,7 +2086,8 @@ public int getAsInt() {
} else if (alterClause instanceof BuildIndexClause) {
BuildIndexClause buildIndexClause = (BuildIndexClause) alterClause;
IndexDef indexDef = buildIndexClause.getIndexDef();
if (indexDef.isInvertedIndex()) {
if (indexDef.getIndexType() == IndexDef.IndexType.NGRAM_BF
|| indexDef.getIndexType() == IndexDef.IndexType.BLOOMFILTER) {
throw new DdlException("ngram bloomfilter or bloomfilter index is not needed to build.");
}
Index index = buildIndexClause.getIndex();
Expand Down
Loading