You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The API doesn't currently allow creating streaming results. It should.
Details:
The run and start methods of DuckDBPreparedStatement return DuckDBPendingResults that are in non-streaming mode, because they use duckdb_execute_prepared and duckdb_pending_prepared, not duckdb_execute_prepared_streaming or duckdb_pending_prepared_streaming. The latter have "DEPRECATION NOTICE"s, which means they will likely be removed in the future, but it's likely they will be replace by some other way to create streaming results. So, they can be used with relatively low risk that the functionality will go away.
Probably DuckDBPreparedStatement should have new methods, stream and startStream, that use duckdb_execute_prepared_streaming and duckdb_pending_prepared_streaming to return DuckDBPendingResults that are in streaming mode.
There should also be an isStreaming (or streaming) boolean property on DuckDBResult that uses duckdb_result_is_streaming.
There's duckdb_stream_fetch_chunk function, but there seems to be no point in exposing it, because it just calls duckdb_fetch_chunk (after checking whether the argument is in streaming mode).
The text was updated successfully, but these errors were encountered:
The API doesn't currently allow creating streaming results. It should.
Details:
The
run
andstart
methods ofDuckDBPreparedStatement
returnDuckDBPendingResult
s that are in non-streaming mode, because they useduckdb_execute_prepared
andduckdb_pending_prepared
, notduckdb_execute_prepared_streaming
orduckdb_pending_prepared_streaming
. The latter have "DEPRECATION NOTICE"s, which means they will likely be removed in the future, but it's likely they will be replace by some other way to create streaming results. So, they can be used with relatively low risk that the functionality will go away.Probably
DuckDBPreparedStatement
should have new methods,stream
andstartStream
, that useduckdb_execute_prepared_streaming
andduckdb_pending_prepared_streaming
to returnDuckDBPendingResult
s that are in streaming mode.There should also be an
isStreaming
(orstreaming
) boolean property onDuckDBResult
that usesduckdb_result_is_streaming
.There's
duckdb_stream_fetch_chunk
function, but there seems to be no point in exposing it, because it just callsduckdb_fetch_chunk
(after checking whether the argument is in streaming mode).The text was updated successfully, but these errors were encountered: