Alter inverted index

[killme2008]

What problem does the new feature solve?

Since v0.10, the INVERTED INDEX constraint can be used to specify the columns that need the inverted index

See https://docs.greptime.com/reference/sql/create/#inverted-index

But we are unable to add or remove a column from INVERTED INDEX.

For example:

-- add load_15 to inverted indexes--
ALTER TABLE monitor MODIFY COLUMN load_15  SET INVERTED INDEX;
-- remove load_15 from inverted indexes--
ALTER TABLE monitor MODIFY COLUMN load_15  UNSET INVERTED INDEX;

What does the feature do?

As the above said.

Implementation challenges

No response

[lyang24]

can i take this issue? is the implementation similar to alter table column with fulltext option like 4952?

[CookiePieWw]

can i take this issue? is the implementation similar to alter table column with fulltext option like #4952?

Sure :) It's quite similar. BTW, I‘d like to recommend unifying related structs like:

SetIndex {
  FULLTEXT,
  INVERTED,
}

which is better for further extenstions.

[lyang24]

I believe we need to add to proto for AlterTableExpr as a prerequisite before working on this one. Can i get a review with this pr?
https://github.com/GreptimeTeam/greptime-proto/pull/204/files

[CookiePieWw]

I believe we need to add to proto for AlterTableExpr as a prerequisite before working on this one. Can i get a review with this pr?

We can first use the commit hash in GreptimeTeam/greptime-proto#204, and merge it after the pr in greptimedb is ready for merge.