JanKaul opened a new issue, #75:
URL: https://github.com/apache/iceberg-rust/issues/75

   Iceberg catalogs that make use of a `*.metadata.json` file to store the 
table metadata require the `metadata_location` and the `TableMetadata` of a 
Table to perform an `update_table` operation ([see 
here](https://github.com/JanKaul/iceberg-rust/blob/memory-catalog/crates/iceberg/src/catalog/memory.rs#L176)).
 It would therefore be helpful to include references to the `metadata_location` 
and the `TableMetadata` in the `TableCommit` payload of the `update_table` 
operation.
   
   Something like:
   
   ```rust
   pub struct TableCommit<'t> {
       /// The table ident.
       pub ident: TableIdent,
       /// Metadata file location of the table
       pub metadata_location: &'t str,
       /// Table metadata
       pub table_metadata: &'t TableMetadata,
       /// The requirements of the table.
       ///
       /// Commit will fail if the requirements are not met.
       pub requirements: Vec<TableRequirement>,
       /// The updates of the table.
       pub updates: Vec<TableUpdate>,
   }
   ```


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

Reply via email to