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

   ### What changes were proposed in this pull request?
   
   Fix ColumnAudit command to display column audit information correctly by 
adding auditInfo support to BaseColumn class and all catalog implementations. 
Additionally, add comprehensive unit tests to verify the fix works correctly 
and displays audit information for columns.
   
   ### Why are the changes needed?
   
   - The ColumnAudit command was not displaying audit information for columns 
because BaseColumn class lacked auditInfo field and auditInfo() method 
implementation
   - When ColumnAudit command tried to retrieve column.auditInfo(), it returned 
null instead of actual audit information
   - All catalog implementations (JDBC, Hive, Hudi, Iceberg, Paimon) inherited 
from BaseColumn but didn't support audit information for columns
   - This created inconsistency where tables and schemas had audit information 
but columns did not
   
   Fix: #8208
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes, this is a functional improvement. The ColumnAudit command now properly 
displays audit information for columns, showing creator, creation time, and 
other audit details that were previously missing.
   
   ### How was this patch tested?
   
   - Existing unit tests continue to pass, ensuring no functional regression
   - Added new comprehensive test case TestColumnAudit to verify column audit 
functionality works correctly
   - Updated all catalog implementation tests to include auditInfo field in 
their builders
   - Local testing completed successfully with Gradle build and Spotless 
formatting
   - All core, clients:cli, and catalog modules compile and test successfully


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