Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: jeremyhi <[email protected]>
  • Loading branch information
CookiePieWw and fengjiachun committed Nov 19, 2024
1 parent f43a2ab commit 82d3d5f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/store-api/src/metadata.rs
Original file line number Diff line number Diff line change
Expand Up @@ -691,7 +691,7 @@ impl RegionMetadataBuilder {
options.is_some(),
InvalidColumnOptionSnafu {
column_name,
msg: "FULLTEXT index options must be provided".to_string(),
msg: "FULLTEXT index options must be provided",
}
);
set_column_fulltext_options(
Expand Down Expand Up @@ -922,7 +922,7 @@ fn unset_column_fulltext_options(
} else {
return InvalidColumnOptionSnafu {
column_name,
msg: "FULLTEXT index already disabled".to_string(),
msg: "FULLTEXT index already disabled",
}
.fail();
}
Expand Down
4 changes: 2 additions & 2 deletions src/table/src/metadata.rs
Original file line number Diff line number Diff line change
Expand Up @@ -292,7 +292,7 @@ impl TableMeta {
options.is_some(),
error::InvalidColumnOptionSnafu {
column_name,
msg: "FULLTEXT index options must be provided".to_string(),
msg: "FULLTEXT index options must be provided",
}
);
set_column_fulltext_options(
Expand Down Expand Up @@ -1012,7 +1012,7 @@ fn set_column_fulltext_options(
!current_options.enable,
error::InvalidColumnOptionSnafu {
column_name,
msg: "FULLTEXT index already enabled".to_string(),
msg: "FULLTEXT index already enabled",
}
);

Expand Down

0 comments on commit 82d3d5f

Please sign in to comment.