feat: write engineInfo with delta-rs version

[zachschuermann]

Description

Write "engineInfo": "delta-rs:0.26.2" (or whatever current version is) in commitInfo action during writes. This will generally assist in debugging to understand what version of delta-rs wrote to tables at different points in time.

Related Issue(s)

Documentation

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.42%. Comparing base (e5971cf) to head (3f919d1).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3584      +/-   ##
==========================================
- Coverage   74.45%   74.42%   -0.04%     
==========================================
  Files         146      146              
  Lines       44616    44617       +1     
  Branches    44616    44617       +1     
==========================================
- Hits        33221    33207      -14     
- Misses       9250     9251       +1     
- Partials     2145     2159      +14     

☔ 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[roeap]

LGTM!

[ion-elgreco]

Hmm we already set it actually here

        app_metadata.insert(
            "clientVersion".to_string(),
            Value::String(format!("delta-rs.{}", crate_version())),

[roeap]

@ion-elgreco - i was sure we had it, but somehow missed it. I'll consolidate in a follow-up.

[zachschuermann]

hey sorry for missing that! thanks @ion-elgreco!

I was still hoping for this being specifically in commitInfo.engineInfo since some existing implementations already parse the string in that field and expose it in e.g. DESCRIBE HISTORY

[ion-elgreco]

hey sorry for missing that! thanks @ion-elgreco!

I was still hoping for this being specifically in commitInfo.engineInfo since some existing implementations already parse the string in that field and expose it in e.g. DESCRIBE HISTORY

We can also keep it there and remove it from appmetadata, but then I hope people are not relying on this info there for other reasons