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

   ### What changes were proposed in this pull request?
   
   - Quote both column names in StarRocks rename column SQL.
   - Skip empty delete column clauses, and return an empty SQL when no change 
remains so no ALTER TABLE is sent.
   
   ### Why are the changes needed?
   
   Unquoted names break renames of special or reserved column names, and an 
empty `ALTER TABLE` is executed by StarRocks as a table OPTIMIZE.
   
   Fix: #14062, #14063
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Added unit tests in `TestStarRocksTableOperationsSqlGeneration`.
   


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