chore: make Transaction cloneable

[dentiny]

What changes are included in this PR?

Iceberg supports retry on commit: https://iceberg.apache.org/docs/latest/configuration/#table-behavior-properties
To implement retry, I would like to commit on a cloned transaction.

Old code without retry:

let action = txn.fast_append();
let action = action.add_data_files(data_file_import_result.new_iceberg_data_files);
txn = action.apply(txn)?;
txn.commit(catalog)

New code with retry:

// Apply all changes.
let action = txn.fast_append();
let action = action.add_data_files(data_file_import_result.new_iceberg_data_files);
txn = action.apply(txn)?;

// Perform commit with retry.
while count < max_retry && time < max_timeout {
  let txn_copy = txn.clone();
  txn.commit(catalog) // commit takes `mut self`, have to make a clone
}

Are these changes tested?

No op change, but compilation and existing tests all pass.

[ZENOTME]

I think the retry commit is implement in the commit internally. For now, looks like we can't guarantee that the transaction can be retry again directly in semantics.🤔

[liurenjie1024]

For now, looks like we can't guarantee that the transaction can be retry again directly in semantics.🤔

Why? Do you mean the commit method consumed transaction?

[liurenjie1024]

Thanks @dentiny for this pr. I'm fine with the changes, but I want to claeify that tx commit is under development, and should happend in commit method internally.

[dentiny]

Thanks @dentiny for this pr. I'm fine with the changes, but I want to claeify that tx commit is under development, and should happend in commit method internally.

Yeah I totally understand.
Thank you for the quick review!