[#9816] fix(catalogs): Fix alter JDBC catalogs column default value problem(cherry-pick)

[yuqi1129]

What changes were proposed in this pull request?

This pull request enhances the handling of default values for string-type columns in JDBC catalog implementations for MySQL, PostgreSQL, Doris, OceanBase, and StarRocks. It ensures that string default values are always properly quoted in generated SQL, addressing cases where default values may be empty or unquoted. Comprehensive unit tests have been added for each implementation to verify this behavior.

Why are the changes needed?

It's a bug.

Fix: #9816

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

UTs.

[Copilot]

Pull request overview

This PR fixes a bug in JDBC catalog implementations where setting empty string default values for columns could cause SQL syntax errors. The fix introduces a check in JdbcColumnDefaultValueConverter to properly quote empty/blank string values for numeric types, and refactors default value handling across MySQL, PostgreSQL, Doris, OceanBase, and StarRocks catalogs.

Changes:

• Added handling for empty/blank default values in JdbcColumnDefaultValueConverter by wrapping them in quotes for numeric types
• Refactored default value SQL generation into a shared appendDefaultValue method in JdbcTableOperations
• Updated all five JDBC catalog implementations to use the new shared method
• Added unit tests for each catalog to verify empty string and non-empty string default value handling

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
catalogs/catalog-jdbc-common/src/main/java/org/apache/gravitino/catalog/jdbc/converter/JdbcColumnDefaultValueConverter.java	Added check to wrap empty/blank values in quotes for numeric types to prevent SQL syntax errors
catalogs/catalog-jdbc-common/src/main/java/org/apache/gravitino/catalog/jdbc/operation/JdbcTableOperations.java	Extracted common default value append logic into new protected method
catalogs/catalog-jdbc-mysql/src/main/java/org/apache/gravitino/catalog/mysql/operation/MysqlTableOperations.java	Refactored to use shared appendDefaultValue method
catalogs/catalog-jdbc-postgresql/src/main/java/org/apache/gravitino/catalog/postgresql/operation/PostgreSqlTableOperations.java	Refactored to use shared appendDefaultValue method, removed unused import
catalogs/catalog-jdbc-doris/src/main/java/org/apache/gravitino/catalog/doris/operation/DorisTableOperations.java	Refactored to use shared appendDefaultValue method, made appendNecessaryProperties visible for testing
catalogs/catalog-jdbc-oceanbase/src/main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseTableOperations.java	Refactored to use shared appendDefaultValue method
catalogs/catalog-jdbc-starrocks/src/main/java/org/apache/gravitino/catalog/starrocks/operations/StarRocksTableOperations.java	Refactored to use shared appendDefaultValue method
catalogs/catalog-jdbc-mysql/src/test/java/org/apache/gravitino/catalog/mysql/operation/TestMysqlTableOperationsSqlGeneration.java	Added tests for empty and non-empty string default values
catalogs/catalog-jdbc-postgresql/src/test/java/org/apache/gravitino/catalog/postgresql/operation/TestPostgreSqlTableOperationsSqlGeneration.java	Added tests for empty and non-empty string default values
catalogs/catalog-jdbc-doris/src/test/java/org/apache/gravitino/catalog/doris/operation/TestDorisTableOperationsSqlGeneration.java	Added tests for empty and non-empty string default values
catalogs/catalog-jdbc-oceanbase/src/test/java/org/apache/gravitino/catalog/oceanbase/operation/TestOceanBaseTableOperationsSqlGeneration.java	Added tests for empty and non-empty string default values
catalogs/catalog-jdbc-starrocks/src/test/java/org/apache/gravitino/catalog/starrocks/operation/TestStarRocksTableOperationsSqlGeneration.java	Added tests for empty, non-empty, and whitespace string default values
catalogs/catalog-jdbc-doris/build.gradle.kts	Added mockito-core test dependency for mocking in tests

[jerryshao]

Can this be merged? @yuqi1129

[yuqi1129]

Can this be merged? @yuqi1129

Yeah, this is just a cherry-pick PR, and it has been merged into main.