yuanoOo commented on code in PR #5228:
URL: https://github.com/apache/gravitino/pull/5228#discussion_r1815883261
##########
catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseTableOperations.java:
##########
@@ -93,18 +274,447 @@ protected String generatePurgeTableSql(String tableName) {
@Override
public void alterTable(String databaseName, String tableName, TableChange...
changes)
throws NoSuchTableException {
- throw new UnsupportedOperationException("Not implemented yet.");
+ LOG.info("Attempting to alter table {} from database {}", tableName,
databaseName);
Review Comment:
Yes, OB does not support some multiple changes in one statement, such as the
following:
```
ALTER TABLE test
ADD COLUMN pro_id INT PRIMARY key,
ADD COLUMN order_id int;
ALTER TABLE `auto_increment_table`
ADD COLUMN `col_6` bigint AUTO_INCREMENT NOT NULL COMMENT 'id' ,
ADD PRIMARY KEY (`col_6`);
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
