databend
databend copied to clipboard
refact: unify the update of table/stream
Sometimes we use `UpdateTableMetaReq` to update a stream, and other times we use `UpdateTableMetaReq.update_stream_meta`. But both `SchemaApi::update_table_meta()` and `update_table_meta()` handle `stream` and `table` with subtle differences. Although the current implementation is logically correct, we could consider refactoring in the future to express the requests for updating a stream and updating a table with a more clear structure, to avoid potential bugs.
Originally posted by @SkyFan2002 in https://github.com/datafuselabs/databend/pull/15582#discussion_r1606390653