Skip to content

Conversation

@otegami
Copy link
Owner

@otegami otegami commented Feb 10, 2025

No description provided.

suketa and others added 30 commits January 18, 2025 07:29
refs: suketaGH-690

In this PR, we add DuckDB::LogicalType#type, which returns logical
type's type.
- [duckdb_type duckdb_get_type_id(duckdb_logical_type type);](https://duckdb.org/docs/api/c/api.html#duckdb_get_type_id)

This is one of the steps for supporting duckdb_logical_column_type
C API.
get error from duckdb when Appender#append_int32 failed.
- fix indent
- fix call-seq
get error message from duckdb when DuckDB::Appender#append_uint8 failed.
suketa and others added 24 commits January 26, 2025 09:40
change error handling of DuckDB::Appender#append_uint8
Try to fix CI failures of suketa#845 by close and reopen database each test.
refs: suketaGH-690

In this PR, we add DuckDB::LogicalType#child_type which is the
following binding method.

```c
duckdb_logical_type duckdb_list_type_child_type(duckdb_logical_type type);
```

ref: https://duckdb.org/docs/api/c/api.html#duckdb_list_type_child_type
ref: https://github.com/duckdb/duckdb/blob/main/src/main/capi/logical_types-c.cpp#L226-L236

This is one of the steps for supporting duckdb_logical_column_type C API.
…ical-type

logical_type: add `DuckDB::LogicalType#child_type`
- CI checks with duckdb 1.2.0.
- change duckdb 1.2.0 in Dockerfile.
DuckDB::Result#row_count, DuckDB::Result#row_size are deprecated.
Because duckdb_row_count C-API of duckdb has been removed and may be
removed in future versions of DuckDB.
…_count

DuckDB::Result#row_count is depreacted
Thease methods don't effect any behavior changes of ruby-duckdb.
…each

deprecate DuckDB::Result.use_chunk_each, DuckDB::Result.use_chunk_each?
use duckdb_fetch_chunk because ruby-duckdb supports duckdb >= 1.0.0.
- calling duckdb_destroy_prepare.
- rescue StandardError.
This reverts commit 504a919.
@otegami otegami force-pushed the test-try-ruby-asan branch 2 times, most recently from 5a65638 to b8efa79 Compare February 10, 2025 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants