refactor!: migrate Snapshot::try_new_from into SnapshotBuilder

[zachschuermann]

Important

This PR is split into two commits: (1) material changes, and (2) callsite updates

What changes are proposed in this pull request?

This PR integrates Snapshot::try_new_from into SnapshotBuilder. We move Snapshot::try_new_from into Snapshot::builder_from and modify SnapshotBuilder to return an Arc<Snapshot> instead of a Snapshot.

Also, Snapshot::builder(table_root) is renamed to Snapshot::builder_for(table_root)

This PR affects the following public APIs

1. Replaces Snapshot::try_new_from(existing) with Snapshot::build_from(existing).build()
2. SnapshotBuilder::build() now returns an Arc<Snapshot> instead of a Snapshot
3. Renames Snapshot::builder to Snapshot::builder_for

How was this change tested?

refactor, updated tests

[codecov]

Codecov Report

❌ Patch coverage is 75.42373% with 29 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.96%. Comparing base (3292cef) to head (95ba8b9).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
kernel/src/snapshot.rs	85.00%	0 Missing and 9 partials ⚠️
kernel/src/checkpoint/tests.rs	40.00%	0 Missing and 6 partials ⚠️
kernel/src/snapshot/builder.rs	80.95%	2 Missing and 2 partials ⚠️
acceptance/src/data.rs	0.00%	0 Missing and 2 partials ⚠️
acceptance/src/meta.rs	0.00%	0 Missing and 2 partials ⚠️
kernel/src/table_changes/mod.rs	71.42%	0 Missing and 2 partials ⚠️
test-utils/src/lib.rs	0.00%	0 Missing and 2 partials ⚠️
ffi/src/lib.rs	50.00%	0 Missing and 1 partial ⚠️
ffi/src/transaction/mod.rs	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1289      +/-   ##
==========================================
- Coverage   83.98%   83.96%   -0.02%     
==========================================
  Files         111      111              
  Lines       26175    26207      +32     
  Branches    26175    26207      +32     
==========================================
+ Hits        21983    22005      +22     
- Misses       3098     3104       +6     
- Partials     1094     1098       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[nicklan]

generally looks good, but will wait for the rename to stamp

[nicklan]

lgtm!