feat: transactional UpdateProperties method support

Author: shuxu.li

src/iceberg/base_transaction.h

@@ -79,9 +79,9 @@ class ICEBERG_EXPORT BaseTransaction : public Transaction {
  private:
   struct TransactionContext {
     TransactionContext() = default;
-    TransactionContext(const TableIdentifier& identifier,
+    TransactionContext(TableIdentifier identifier,
                        std::shared_ptr<TableMetadata> metadata)
-        : identifier(identifier), current_metadata(std::move(metadata)) {}
+        : identifier(std::move(identifier)), current_metadata(std::move(metadata)) {}
 
     TransactionContext(const TransactionContext&) = delete;
     TransactionContext& operator=(const TransactionContext&) = delete;

src/iceberg/test/base_transaction_test.cc

@@ -102,7 +102,7 @@ TEST_F(BaseTransactionTest, RemovePropertiesSkipsMissingKeys) {
             dynamic_cast<const table::RemoveProperties*>(updates.front().get());
         EXPECT_NE(remove_update, nullptr);
         EXPECT_THAT(remove_update->removed(),
-                    ::testing::ElementsAre("existing", "missing"));
+                    ::testing::UnorderedElementsAre("missing", "existing"));
         return {std::unique_ptr<Table>()};
       });