[westonpace]

We need to ensure all of our exec nodes do not spawn tasks in the execute fn until apache/datafusion#16312 is addressed. The AddRowAddrExec was violating this and this PR fixes that.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 89.79592% with 5 lines in your changes missing coverage. Please review.

Project coverage is 78.40%. Comparing base (fc3c8d4) to head (ce0a529).
Report is 6 commits behind head on main.

Files with missing lines	Patch %	Lines
rust/lance/src/io/exec/rowids.rs	89.79%	0 Missing and 5 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3954      +/-   ##
==========================================
+ Coverage   78.35%   78.40%   +0.04%     
==========================================
  Files         285      285              
  Lines      110238   111457    +1219     
  Branches   110238   111457    +1219     
==========================================
+ Hits        86378    87385    +1007     
- Misses      20482    20683     +201     
- Partials     3378     3389      +11     

Flag	Coverage Δ
unittests	78.40% <89.79%> (+0.04%)	⬆️

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.

[renato2099]

thanks for fixing this @westonpace !