[GitHub] [iceberg] RussellSpitzer commented on issue #8331: Improper commit message post table migration at the Nessie commit

["RussellSpitzer (via GitHub)" <gi...@apache.org>]

RussellSpitzer commented on issue #8331:
URL: https://github.com/apache/iceberg/issues/8331#issuecomment-1680400541

   Register table does not create a new commit as far as I know. You are looking at the last commit to the table Sent from my iPhoneOn Aug 16, 2023, at 4:10 AM, Harsh Maheshwari ***@***.***> wrote:
   Apache Iceberg version
   1.3.1 (latest release)
   Query engine
   Spark
   Please describe the bug 🐞
   Steps to repro:
   
   
   Migrate a table with existing snapshots on to the NessieCatalog using registerTable(..)
   
   
   The commit message is configured against the commit type of the current snapshot
   
   
   Iceberg append against db.migration_table
   Authored by: xx 2 minutes ago
   
   Properties:
   application-type: iceberg
   iceberg.operation: append
   
   https://github.com/apache/iceberg/blob/c1fc55022557653cfeb6125c1d51bf8ab12c8f96/nessie/src/main/java/org/apache/iceberg/nessie/NessieIcebergClient.java#L475C15-L475C28
   The ask of this ticket is for a better commit message and iceberg.operation, which clearly identifies that it's a table registration activity.
   
   —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>


-- 
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: issues-unsubscribe@iceberg.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org