CFSQL-1473: fix D1 exec throwing TypeError on invalid SQL queries

[FlorentCollin]

We were trying to call addAggregatedD1MetaToSpan before checking if any of the results are errors in a .exec call. When an error is returned the meta property is not set.

This commit is now checking the errors first and then call the addAggregatedD1MetaToSpan if there is no error.

[codspeed-hq]

Merging this PR will not alter performance

✅ 70 untouched benchmarks
⏩ 129 skipped benchmarks¹

---

_{Comparing fcollin/CFSQL-1473-fix-db-exec-type-error (297101f) with main (02f8f80)}

Footnotes

1. 129 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[github-actions]

The generated output of @cloudflare/workers-types matches the snapshot in types/generated-snapshot 🎉

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.36%. Comparing base (02f8f80) to head (297101f).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6029      +/-   ##
==========================================
+ Coverage   70.34%   70.36%   +0.01%     
==========================================
  Files         408      408              
  Lines      108646   108646              
  Branches    17991    17991              
==========================================
+ Hits        76431    76444      +13     
+ Misses      21412    21402      -10     
+ Partials    10803    10800       -3     

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