emilie-wang opened a new pull request, #575:
URL: https://github.com/apache/incubator-xtable/pull/575

   
   ## *Important Read*
   
   Aim to address https://github.com/apache/incubator-xtable/issues/525
   
   ## What is the purpose of the pull request
   
   It is expected to capture the comments and sync the comments of field when 
syncing schema. The PR makes the comments of schema fields are captured and 
synced correctly.
   
   ## Brief change log
   
   - For Delta target, extract the comment from internal schema and add to 
Spark schema representative. 
   - For Iceberg target, make sure any difference of comments is captured 
during syncSchema() and the schema is updated with new comments.
   
   ## Verify this pull request
   
   This change added tests and can be verified as follows:
   
   - *Added unit tests to test the comments in Iceberg schema sync test class*
   - *Added comments in Delta schema extractor test class*
   


-- 
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: commits-unsubscr...@xtable.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to