diqiu50 opened a new pull request, #14057:
URL: https://github.com/apache/gravitino/pull/14057

   ### What changes were proposed in this pull request?
   
   Remove the NULL-default check in `validateUpdateColumnNullable`, and drop 
the NULL default when regenerating the column definition for MySQL, OceanBase, 
Doris and ClickHouse.
   
   ### Why are the changes needed?
   
   A nullable column with no default is loaded with `Literals.NULL` as its 
default, so it could not be changed to NOT NULL.
   
   Fix: #14056
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. A nullable column with no default can now be changed to NOT NULL on 
JDBC catalogs.
   
   ### How was this patch tested?
   
   Updated unit tests for MySQL, PostgreSQL, OceanBase and ClickHouse.
   


-- 
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]

Reply via email to