duckdb 1.1.3.0 → 1.2.0.0

data/ext/duckdb/appender.c

@@ -6,14 +6,13 @@ static void deallocate(void *);
 static VALUE allocate(VALUE klass);
 static size_t memsize(const void *p);
 static VALUE appender_initialize(VALUE klass, VALUE con, VALUE schema, VALUE table);
-static VALUE appender_begin_row(VALUE self);
-static VALUE appender_end_row(VALUE self);
-static VALUE appender_append_bool(VALUE self, VALUE val);
-static VALUE appender_append_int8(VALUE self, VALUE val);
-static VALUE appender_append_int16(VALUE self, VALUE val);
-static VALUE appender_append_int32(VALUE self, VALUE val);
-static VALUE appender_append_int64(VALUE self, VALUE val);
-static VALUE appender_append_uint8(VALUE self, VALUE val);
+static VALUE appender_error_message(VALUE self);
+static VALUE appender__append_bool(VALUE self, VALUE val);
+static VALUE appender__append_int8(VALUE self, VALUE val);
+static VALUE appender__append_int16(VALUE self, VALUE val);
+static VALUE appender__append_int32(VALUE self, VALUE val);
+static VALUE appender__append_int64(VALUE self, VALUE val);
+static VALUE appender__append_uint8(VALUE self, VALUE val);
 static VALUE appender_append_uint16(VALUE self, VALUE val);
 static VALUE appender_append_uint32(VALUE self, VALUE val);
 static VALUE appender_append_uint64(VALUE self, VALUE val);
@@ -23,14 +22,21 @@ static VALUE appender_append_varchar(VALUE self, VALUE val);
 static VALUE appender_append_varchar_length(VALUE self, VALUE val, VALUE len);
 static VALUE appender_append_blob(VALUE self, VALUE val);
 static VALUE appender_append_null(VALUE self);
+
+#ifdef HAVE_DUCKDB_H_GE_V1_1_0
+static VALUE appender_append_default(VALUE self);
+#endif
+
+static VALUE appender__end_row(VALUE self);
 static VALUE appender__append_date(VALUE self, VALUE yearval, VALUE monthval, VALUE dayval);
 static VALUE appender__append_interval(VALUE self, VALUE months, VALUE days, VALUE micros);
 static VALUE appender__append_time(VALUE self, VALUE hour, VALUE min, VALUE sec, VALUE micros);
 static VALUE appender__append_timestamp(VALUE self, VALUE year, VALUE month, VALUE day, VALUE hour, VALUE min, VALUE sec, VALUE micros);
 static VALUE appender__append_hugeint(VALUE self, VALUE lower, VALUE upper);
 static VALUE appender__append_uhugeint(VALUE self, VALUE lower, VALUE upper);
-static VALUE appender_flush(VALUE self);
-static VALUE appender_close(VALUE self);
+static VALUE appender__flush(VALUE self);
+static VALUE appender__close(VALUE self);
+static VALUE duckdb_state_to_bool_value(duckdb_state state);
 
 static const rb_data_type_t appender_data_type = {
     "DuckDB/Appender",
@@ -77,27 +83,40 @@ static VALUE appender_initialize(VALUE self, VALUE con, VALUE schema, VALUE tabl
     return self;
 }
 
-static VALUE appender_begin_row(VALUE self) {
+/* call-seq:
+ *   appender.error_message -> String
+ *
+ * Returns the error message of the appender. If there is no error, then it returns nil.
+ *
+ *   require 'duckdb'
+ *   db = DuckDB::Database.open
+ *   con = db.connect
+ *   con.query('CREATE TABLE users (id INTEGER, name VARCHAR)')
+ *   appender = con.appender('users')
+ *   appender.error_message # => nil
+ */
+static VALUE appender_error_message(VALUE self) {
     rubyDuckDBAppender *ctx;
+    const char *msg;
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
-    if (duckdb_appender_begin_row(ctx->appender) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to flush");
+    msg = duckdb_appender_error(ctx->appender);
+    if (msg == NULL) {
+        return Qnil;
     }
-    return self;
+    return rb_str_new2(msg);
 }
 
-static VALUE appender_end_row(VALUE self) {
+/* :nodoc: */
+static VALUE appender__end_row(VALUE self) {
     rubyDuckDBAppender *ctx;
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
-    if (duckdb_appender_end_row(ctx->appender) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to flush");
-    }
-    return self;
+    return duckdb_state_to_bool_value(duckdb_appender_end_row(ctx->appender));
 }
 
-static VALUE appender_append_bool(VALUE self, VALUE val) {
+/* :nodoc: */
+static VALUE appender__append_bool(VALUE self, VALUE val) {
     rubyDuckDBAppender *ctx;
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
@@ -105,70 +124,57 @@ static VALUE appender_append_bool(VALUE self, VALUE val) {
         rb_raise(rb_eArgError, "argument must be boolean");
     }
 
-    if (duckdb_append_bool(ctx->appender, (val == Qtrue)) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append boolean");
-    }
-    return self;
+    return duckdb_state_to_bool_value(duckdb_append_bool(ctx->appender, (val == Qtrue)));
 }
 
-static VALUE appender_append_int8(VALUE self, VALUE val) {
+/* :nodoc: */
+static VALUE appender__append_int8(VALUE self, VALUE val) {
     rubyDuckDBAppender *ctx;
     int8_t i8val = (int8_t)NUM2INT(val);
 
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
-    if (duckdb_append_int8(ctx->appender, i8val) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
-    }
-    return self;
+    return duckdb_state_to_bool_value(duckdb_append_int8(ctx->appender, i8val));
 }
 
-static VALUE appender_append_int16(VALUE self, VALUE val) {
+/* :nodoc: */
+static VALUE appender__append_int16(VALUE self, VALUE val) {
     rubyDuckDBAppender *ctx;
     int16_t i16val = (int16_t)NUM2INT(val);
 
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
-    if (duckdb_append_int16(ctx->appender, i16val) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
-    }
-    return self;
+    return duckdb_state_to_bool_value(duckdb_append_int16(ctx->appender, i16val));
 }
 
-static VALUE appender_append_int32(VALUE self, VALUE val) {
+/* :nodoc: */
+static VALUE appender__append_int32(VALUE self, VALUE val) {
     rubyDuckDBAppender *ctx;
     int32_t i32val = (int32_t)NUM2INT(val);
 
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
-    if (duckdb_append_int32(ctx->appender, i32val) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
-    }
-    return self;
+    return duckdb_state_to_bool_value(duckdb_append_int32(ctx->appender, i32val));
 }
 
-static VALUE appender_append_int64(VALUE self, VALUE val) {
+/* :nodoc: */
+static VALUE appender__append_int64(VALUE self, VALUE val) {
     rubyDuckDBAppender *ctx;
     int64_t i64val = (int64_t)NUM2LL(val);
 
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
-    if (duckdb_append_int64(ctx->appender, i64val) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
-    }
-    return self;
+    return duckdb_state_to_bool_value(duckdb_append_int64(ctx->appender, i64val));
 }
 
-static VALUE appender_append_uint8(VALUE self, VALUE val) {
+/* :nodoc: */
+static VALUE appender__append_uint8(VALUE self, VALUE val) {
     rubyDuckDBAppender *ctx;
-    int8_t ui8val = (uint8_t)NUM2UINT(val);
+    uint8_t ui8val = (uint8_t)NUM2UINT(val);
 
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
-    if (duckdb_append_uint8(ctx->appender, ui8val) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
-    }
-    return self;
+    return duckdb_state_to_bool_value(duckdb_append_uint8(ctx->appender, ui8val));
 }
 
 static VALUE appender_append_uint16(VALUE self, VALUE val) {
@@ -178,7 +184,7 @@ static VALUE appender_append_uint16(VALUE self, VALUE val) {
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
     if (duckdb_append_uint16(ctx->appender, ui16val) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
+        rb_raise(eDuckDBError, "failed to append uint16");
     }
     return self;
 }
@@ -190,7 +196,7 @@ static VALUE appender_append_uint32(VALUE self, VALUE val) {
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
     if (duckdb_append_uint32(ctx->appender, ui32val) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
+        rb_raise(eDuckDBError, "failed to append uint32");
     }
     return self;
 }
@@ -202,7 +208,7 @@ static VALUE appender_append_uint64(VALUE self, VALUE val) {
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
     if (duckdb_append_uint64(ctx->appender, ui64val) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
+        rb_raise(eDuckDBError, "failed to append uint64");
     }
     return self;
 }
@@ -214,7 +220,7 @@ static VALUE appender_append_float(VALUE self, VALUE val) {
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
     if (duckdb_append_float(ctx->appender, fval) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
+        rb_raise(eDuckDBError, "failed to append float");
     }
     return self;
 }
@@ -226,7 +232,7 @@ static VALUE appender_append_double(VALUE self, VALUE val) {
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
     if (duckdb_append_double(ctx->appender, dval) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
+        rb_raise(eDuckDBError, "failed to append double");
     }
     return self;
 }
@@ -238,7 +244,7 @@ static VALUE appender_append_varchar(VALUE self, VALUE val) {
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
     if (duckdb_append_varchar(ctx->appender, pval) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
+        rb_raise(eDuckDBError, "failed to append varchar");
     }
     return self;
 }
@@ -252,7 +258,7 @@ static VALUE appender_append_varchar_length(VALUE self, VALUE val, VALUE len) {
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
     if (duckdb_append_varchar_length(ctx->appender, pval, length) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
+        rb_raise(eDuckDBError, "failed to append varchar with length");
     }
     return self;
 }
@@ -266,7 +272,7 @@ static VALUE appender_append_blob(VALUE self, VALUE val) {
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
     if (duckdb_append_blob(ctx->appender, (void *)pval, length) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
+        rb_raise(eDuckDBError, "failed to append blob");
     }
     return self;
 }
@@ -276,11 +282,24 @@ static VALUE appender_append_null(VALUE self) {
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
     if (duckdb_append_null(ctx->appender) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to append");
+        rb_raise(eDuckDBError, "failed to append null");
     }
     return self;
 }
 
+#ifdef HAVE_DUCKDB_H_GE_V1_1_0
+static VALUE appender_append_default(VALUE self) {
+    rubyDuckDBAppender *ctx;
+    TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
+
+    if (duckdb_append_default(ctx->appender) == DuckDBError) {
+        rb_raise(eDuckDBError, "failed to append default");
+    }
+    return self;
+}
+#endif
+
+/* :nodoc: */
 static VALUE appender__append_date(VALUE self, VALUE year, VALUE month, VALUE day) {
     duckdb_date dt;
     rubyDuckDBAppender *ctx;
@@ -294,6 +313,7 @@ static VALUE appender__append_date(VALUE self, VALUE year, VALUE month, VALUE da
     return self;
 }
 
+/* :nodoc: */
 static VALUE appender__append_interval(VALUE self, VALUE months, VALUE days, VALUE micros) {
     duckdb_interval interval;
     rubyDuckDBAppender *ctx;
@@ -307,6 +327,7 @@ static VALUE appender__append_interval(VALUE self, VALUE months, VALUE days, VAL
     return self;
 }
 
+/* :nodoc: */
 static VALUE appender__append_time(VALUE self, VALUE hour, VALUE min, VALUE sec, VALUE micros) {
     duckdb_time time;
     rubyDuckDBAppender *ctx;
@@ -320,6 +341,7 @@ static VALUE appender__append_time(VALUE self, VALUE hour, VALUE min, VALUE sec,
     return self;
 }
 
+/* :nodoc: */
 static VALUE appender__append_timestamp(VALUE self, VALUE year, VALUE month, VALUE day, VALUE hour, VALUE min, VALUE sec, VALUE micros) {
     duckdb_timestamp timestamp;
 
@@ -335,6 +357,7 @@ static VALUE appender__append_timestamp(VALUE self, VALUE year, VALUE month, VAL
     return self;
 }
 
+/* :nodoc: */
 static VALUE appender__append_hugeint(VALUE self, VALUE lower, VALUE upper) {
     duckdb_hugeint hugeint;
 
@@ -350,6 +373,7 @@ static VALUE appender__append_hugeint(VALUE self, VALUE lower, VALUE upper) {
     return self;
 }
 
+/* :nodoc: */
 static VALUE appender__append_uhugeint(VALUE self, VALUE lower, VALUE upper) {
     duckdb_uhugeint uhugeint;
 
@@ -365,38 +389,37 @@ static VALUE appender__append_uhugeint(VALUE self, VALUE lower, VALUE upper) {
     return self;
 }
 
-static VALUE appender_flush(VALUE self) {
+/* :nodoc: */
+static VALUE appender__flush(VALUE self) {
     rubyDuckDBAppender *ctx;
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
-    if (duckdb_appender_flush(ctx->appender) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to flush");
-    }
-    return self;
+    return duckdb_state_to_bool_value(duckdb_appender_flush(ctx->appender));
 }
 
-static VALUE appender_close(VALUE self) {
+/* :nodoc: */
+static VALUE appender__close(VALUE self) {
     rubyDuckDBAppender *ctx;
     TypedData_Get_Struct(self, rubyDuckDBAppender, &appender_data_type, ctx);
 
-    if (duckdb_appender_close(ctx->appender) == DuckDBError) {
-        rb_raise(eDuckDBError, "failed to flush");
+    return duckdb_state_to_bool_value(duckdb_appender_close(ctx->appender));
+}
+
+static VALUE duckdb_state_to_bool_value(duckdb_state state) {
+    if (state == DuckDBSuccess) {
+        return Qtrue;
     }
-    return self;
+    return Qfalse;
 }
 
 void rbduckdb_init_duckdb_appender(void) {
+#if 0
+    VALUE mDuckDB = rb_define_module("DuckDB");
+#endif
     cDuckDBAppender = rb_define_class_under(mDuckDB, "Appender", rb_cObject);
     rb_define_alloc_func(cDuckDBAppender, allocate);
     rb_define_method(cDuckDBAppender, "initialize", appender_initialize, 3);
-    rb_define_method(cDuckDBAppender, "begin_row", appender_begin_row, 0);
-    rb_define_method(cDuckDBAppender, "end_row", appender_end_row, 0);
-    rb_define_method(cDuckDBAppender, "append_bool", appender_append_bool, 1);
-    rb_define_method(cDuckDBAppender, "append_int8", appender_append_int8, 1);
-    rb_define_method(cDuckDBAppender, "append_int16", appender_append_int16, 1);
-    rb_define_method(cDuckDBAppender, "append_int32", appender_append_int32, 1);
-    rb_define_method(cDuckDBAppender, "append_int64", appender_append_int64, 1);
-    rb_define_method(cDuckDBAppender, "append_uint8", appender_append_uint8, 1);
+    rb_define_method(cDuckDBAppender, "error_message", appender_error_message, 0);
     rb_define_method(cDuckDBAppender, "append_uint16", appender_append_uint16, 1);
     rb_define_method(cDuckDBAppender, "append_uint32", appender_append_uint32, 1);
     rb_define_method(cDuckDBAppender, "append_uint64", appender_append_uint64, 1);
@@ -406,12 +429,24 @@ void rbduckdb_init_duckdb_appender(void) {
     rb_define_method(cDuckDBAppender, "append_varchar_length", appender_append_varchar_length, 2);
     rb_define_method(cDuckDBAppender, "append_blob", appender_append_blob, 1);
     rb_define_method(cDuckDBAppender, "append_null", appender_append_null, 0);
+
+#ifdef HAVE_DUCKDB_H_GE_V1_1_0
+    rb_define_method(cDuckDBAppender, "append_default", appender_append_default, 0);
+#endif
+
+    rb_define_private_method(cDuckDBAppender, "_end_row", appender__end_row, 0);
+    rb_define_private_method(cDuckDBAppender, "_flush", appender__flush, 0);
+    rb_define_private_method(cDuckDBAppender, "_close", appender__close, 0);
+    rb_define_private_method(cDuckDBAppender, "_append_bool", appender__append_bool, 1);
+    rb_define_private_method(cDuckDBAppender, "_append_int8", appender__append_int8, 1);
+    rb_define_private_method(cDuckDBAppender, "_append_int16", appender__append_int16, 1);
+    rb_define_private_method(cDuckDBAppender, "_append_int32", appender__append_int32, 1);
+    rb_define_private_method(cDuckDBAppender, "_append_int64", appender__append_int64, 1);
+    rb_define_private_method(cDuckDBAppender, "_append_uint8", appender__append_uint8, 1);
     rb_define_private_method(cDuckDBAppender, "_append_date", appender__append_date, 3);
     rb_define_private_method(cDuckDBAppender, "_append_interval", appender__append_interval, 3);
     rb_define_private_method(cDuckDBAppender, "_append_time", appender__append_time, 4);
     rb_define_private_method(cDuckDBAppender, "_append_timestamp", appender__append_timestamp, 7);
     rb_define_private_method(cDuckDBAppender, "_append_hugeint", appender__append_hugeint, 2);
     rb_define_private_method(cDuckDBAppender, "_append_uhugeint", appender__append_uhugeint, 2);
-    rb_define_method(cDuckDBAppender, "flush", appender_flush, 0);
-    rb_define_method(cDuckDBAppender, "close", appender_close, 0);
 }

data/ext/duckdb/blob.c

@@ -3,5 +3,8 @@
 VALUE cDuckDBBlob;
 
 void rbduckdb_init_duckdb_blob(void) {
+#if 0
+    VALUE mDuckDB = rb_define_module("DuckDB");
+#endif
     cDuckDBBlob = rb_define_class_under(mDuckDB, "Blob", rb_cString);
 }

data/ext/duckdb/column.c

@@ -6,6 +6,7 @@ static void deallocate(void *ctx);
 static VALUE allocate(VALUE klass);
 static size_t memsize(const void *p);
 static VALUE duckdb_column__type(VALUE oDuckDBColumn);
+static VALUE duckdb_column__logical_type(VALUE oDuckDBColumn);
 static VALUE duckdb_column_get_name(VALUE oDuckDBColumn);
 
 static const rb_data_type_t column_data_type = {
@@ -29,9 +30,7 @@ static size_t memsize(const void *p) {
     return sizeof(rubyDuckDBColumn);
 }
 
-/*
- *
- */
+/* :nodoc: */
 VALUE duckdb_column__type(VALUE oDuckDBColumn) {
     rubyDuckDBColumn *ctx;
     rubyDuckDBResult *ctxresult;
@@ -47,6 +46,33 @@ VALUE duckdb_column__type(VALUE oDuckDBColumn) {
     return INT2FIX(type);
 }
 
+/*
+ *  call-seq:
+ *    column.logical_type -> DuckDB::LogicalType
+ *
+ *  Returns the logical type class.
+ *
+ */
+VALUE duckdb_column__logical_type(VALUE oDuckDBColumn) {
+    rubyDuckDBColumn *ctx;
+    rubyDuckDBResult *ctxresult;
+    VALUE result;
+    duckdb_logical_type _logical_type;
+    VALUE logical_type = Qnil;
+
+    TypedData_Get_Struct(oDuckDBColumn, rubyDuckDBColumn, &column_data_type, ctx);
+
+    result = rb_ivar_get(oDuckDBColumn, rb_intern("result"));
+    ctxresult = get_struct_result(result);
+    _logical_type = duckdb_column_logical_type(&(ctxresult->result), ctx->col);
+
+    if (_logical_type) {
+        logical_type = rbduckdb_create_logical_type(_logical_type);
+    }
+
+    return logical_type;
+}
+
 /*
  *  call-seq:
  *    column.name -> string.
@@ -82,9 +108,13 @@ VALUE rbduckdb_create_column(VALUE oDuckDBResult, idx_t col) {
 }
 
 void rbduckdb_init_duckdb_column(void) {
+#if 0
+    VALUE mDuckDB = rb_define_module("DuckDB");
+#endif
     cDuckDBColumn = rb_define_class_under(mDuckDB, "Column", rb_cObject);
     rb_define_alloc_func(cDuckDBColumn, allocate);
 
     rb_define_private_method(cDuckDBColumn, "_type", duckdb_column__type, 0);
+    rb_define_method(cDuckDBColumn, "logical_type", duckdb_column__logical_type, 0);
     rb_define_method(cDuckDBColumn, "name", duckdb_column_get_name, 0);
 }

data/ext/duckdb/config.c

@@ -80,6 +80,9 @@ static VALUE config_set_config(VALUE self, VALUE key, VALUE value) {
 }
 
 void rbduckdb_init_duckdb_config(void) {
+#if 0
+    VALUE mDuckDB = rb_define_module("DuckDB");
+#endif
     cDuckDBConfig = rb_define_class_under(mDuckDB, "Config", rb_cObject);
     rb_define_alloc_func(cDuckDBConfig, allocate);
     rb_define_singleton_method(cDuckDBConfig, "size", config_s_size, 0);

data/ext/duckdb/connection.c

@@ -114,6 +114,7 @@ static VALUE duckdb_connection_query_progress(VALUE self) {
     return rb_funcall(mDuckDBConverter, rb_intern("_to_query_progress"), 3, DBL2NUM(progress.percentage), ULL2NUM(progress.rows_processed), ULL2NUM(progress.total_rows_to_process));
 }
 
+/* :nodoc: */
 static VALUE duckdb_connection_connect(VALUE self, VALUE oDuckDBDatabase) {
     rubyDuckDBConnection *ctx;
     rubyDuckDB *ctxdb;
@@ -131,6 +132,7 @@ static VALUE duckdb_connection_connect(VALUE self, VALUE oDuckDBDatabase) {
     return self;
 }
 
+/* :nodoc: */
 static VALUE duckdb_connection_query_sql(VALUE self, VALUE str) {
     rubyDuckDBConnection *ctx;
     rubyDuckDBResult *ctxr;
@@ -151,6 +153,9 @@ static VALUE duckdb_connection_query_sql(VALUE self, VALUE str) {
 }
 
 void rbduckdb_init_duckdb_connection(void) {
+#if 0
+    VALUE mDuckDB = rb_define_module("DuckDB");
+#endif
     cDuckDBConnection = rb_define_class_under(mDuckDB, "Connection", rb_cObject);
     rb_define_alloc_func(cDuckDBConnection, allocate);
 
@@ -158,5 +163,6 @@ void rbduckdb_init_duckdb_connection(void) {
     rb_define_method(cDuckDBConnection, "interrupt", duckdb_connection_interrupt, 0);
     rb_define_method(cDuckDBConnection, "query_progress", duckdb_connection_query_progress, 0);
     rb_define_private_method(cDuckDBConnection, "_connect", duckdb_connection_connect, 1);
+    /* TODO: query_sql => _query_sql */
     rb_define_private_method(cDuckDBConnection, "query_sql", duckdb_connection_query_sql, 1);
 }

data/ext/duckdb/database.c

@@ -43,6 +43,7 @@ rubyDuckDB *rbduckdb_get_struct_database(VALUE obj) {
     return ctx;
 }
 
+/* :nodoc: */
 static VALUE duckdb_database_s_open(int argc, VALUE *argv, VALUE cDuckDBDatabase) {
     rubyDuckDB *ctx;
     VALUE obj;
@@ -64,6 +65,7 @@ static VALUE duckdb_database_s_open(int argc, VALUE *argv, VALUE cDuckDBDatabase
     return obj;
 }
 
+/* :nodoc: */
 static VALUE duckdb_database_s_open_ext(int argc, VALUE *argv, VALUE cDuckDBDatabase) {
     rubyDuckDB *ctx;
     VALUE obj;
@@ -98,6 +100,7 @@ static VALUE duckdb_database_s_open_ext(int argc, VALUE *argv, VALUE cDuckDBData
     return obj;
 }
 
+/* :nodoc: */
 static VALUE duckdb_database_connect(VALUE self) {
     return rbduckdb_create_connection(self);
 }
@@ -116,6 +119,9 @@ static VALUE duckdb_database_close(VALUE self) {
 }
 
 void rbduckdb_init_duckdb_database(void) {
+#if 0
+    VALUE mDuckDB = rb_define_module("DuckDB");
+#endif
     cDuckDBDatabase = rb_define_class_under(mDuckDB, "Database", rb_cObject);
     rb_define_alloc_func(cDuckDBDatabase, allocate);
     rb_define_singleton_method(cDuckDBDatabase, "_open", duckdb_database_s_open, -1);

data/ext/duckdb/duckdb.c

@@ -31,6 +31,7 @@ Init_duckdb_native(void) {
     rbduckdb_init_duckdb_connection();
     rbduckdb_init_duckdb_result();
     rbduckdb_init_duckdb_column();
+    rbduckdb_init_duckdb_logical_type();
     rbduckdb_init_duckdb_prepared_statement();
     rbduckdb_init_duckdb_pending_result();
     rbduckdb_init_duckdb_blob();

data/ext/duckdb/error.c

@@ -3,5 +3,8 @@
 VALUE eDuckDBError;
 
 void rbduckdb_init_duckdb_error(void) {
+#if 0
+    VALUE mDuckDB = rb_define_module("DuckDB");
+#endif
     eDuckDBError = rb_define_class_under(mDuckDB, "Error", rb_eStandardError);
 }

data/ext/duckdb/extconf.rb

@@ -61,9 +61,6 @@ check_duckdb_library('duckdb', 'duckdb_appender_column_count', DUCKDB_REQUIRED_V
 # check duckdb >= 1.0.0
 have_func('duckdb_fetch_chunk', 'duckdb.h')
 
-# check duckdb >= 1.0.0
-have_func('duckdb_fetch_chunk', 'duckdb.h')
-
 # check duckdb >= 1.1.0
 have_func('duckdb_result_error_type', 'duckdb.h')
 

data/ext/duckdb/extracted_statements.c

@@ -95,6 +95,9 @@ static VALUE duckdb_extracted_statements_prepared_statement(VALUE self, VALUE co
 }
 
 void rbduckdb_init_duckdb_extracted_statements(void) {
+#if 0
+    VALUE mDuckDB = rb_define_module("DuckDB");
+#endif
     cDuckDBExtractedStatements = rb_define_class_under(mDuckDB, "ExtractedStatementsImpl", rb_cObject);
 
     rb_define_alloc_func(cDuckDBExtractedStatements, allocate);