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

   ### What changes were proposed in this pull request?
   
   Quote the table name in four ALTER-statement builders in
   PostgreSqlTableOperations.java that previously emitted it unquoted:
   addColumn, updateColumnDefaultValue, updateColumnType, renameColumn.
   
   ### Why are the changes needed?
   
   Mixed-case PostgreSQL table names (e.g. "MixedCaseTbl") fail to
   alter because the unquoted identifier gets folded to lower case by
   PostgreSQL, so the statement targets a non-existent relation.
   
   Fix: #13024
   
   ### Does this PR introduce any user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Added regression tests covering the four alter paths with a
   mixed-case table name in TestPostgreSqlTableOperationsSqlGeneration.
   


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