Vendoring broken

[krlmlr]

With duckdb/duckdb#12636.

https://github.com/duckdb/duckdb-r/actions/runs/9722578776/job/26836690380#step:8:683

statement.cpp: In function ‘SEXPREC* rapi_execute(duckdb::stmt_eptr_t, bool, bool)’:
statement.cpp:345:39: error: ‘IsFinishedOrBlocked’ is not a member of ‘duckdb::PendingQueryResult’
  345 |         } while (!PendingQueryResult::IsFinishedOrBlocked(execution_result));
      |                                       ^~~~~~~~~~~~~~~~~~~

[krlmlr]

@Tishj: PendingQueryResult::IsFinishedOrBlocked() is gone after duckdb/duckdb#12636, is it safe to replace with PendingQueryResult::IsFinished() ?

In #90, I introduced PendingQueryResult::IsFinishedOrBlocked() that seemed necessary back then, cherry-picking a commit by @Tmonster: #90 (comment) .

Thanks!

[Tishj]

@krlmlr Yes, that change was introduced by duckdb/duckdb#12636
You should now use PendingQueryResult::IsResultReady no matter if the result collector is materialized or streaming

I should have mentioned that in the PR description

[krlmlr]

Fixed now, thanks!