fix: remove unnecessary RETURNING * from database adapter update

[wschurman]

Why

When reviewing #540, I noticed that the RETURNING * isn't necessary, even in the normal case (since #233).

How

Remove the RETURNING *, return a simple updatedRowCount instead for error behavior.

Test Plan

Run tests.

[wschurman]

• fix: remove unnecessary RETURNING * from database adapter update #555 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (4709f53) to head (50f5f25).

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #555   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          110       110           
  Lines        17588     17583    -5     
  Branches      1515      1515           
=========================================
- Hits         17588     17583    -5     

Flag	Coverage Δ
integration	28.35% <33.33%> (+0.01%)	⬆️
unittest	95.19% <66.66%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[ide]

This docblock needs to change.