dttung2905 opened a new pull request, #333: URL: https://github.com/apache/iceberg-go/pull/333
Hi team, I made a dumb mistake in https://github.com/apache/iceberg-go/pull/326 that the table created is a standard AWS Glue Table.It should ,however, be Apache Iceberg-type table instead. I'm really sorry for that :( After `CreateTable` method is called, AWS Glue will create a brand new blob to store the metadata file under the format of `${metadataLocation}/metadata/00000-00000-00000-00000-00000.metadata.json`. If user already has an existing metadata file in S3, `RegisterTable` method should be used instead, similar to `RegisterTable` in REST catalog https://github.com/apache/iceberg-go/blob/9572985db3913f2a9c5c711e5a40cd61769c21aa/catalog/rest/rest.go#L806-L830 Also, let me know if we want to drop `TestGlueCreateTableSuccessIntegration` for now because I think it is quite troublesome to perform clean up and make sure there is no dangling resources left after the test -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
