clickhouse-native 0.1.3 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 99a3891e30d35144bd50c299fe5ee814e4a491860c7a7d066043982563960c39
-  data.tar.gz: 3c0dbaff2bc6db96ff5e1a8c5dbab67d6b4233571834e127f5a115570ef70346
+  metadata.gz: 1754175716e40a4b42c958e5234273724e03875eb1e466d8dd8d9855d45f6b6e
+  data.tar.gz: 28c8d5d65fbf242476d8dc6ffd2f5a53b93f920b5087955757530e5e0fb2dd6e
 SHA512:
-  metadata.gz: b16891dc09f48773e9bdb59fffa07a645c4ab90dbc6723f58c48369eb18d81995f7df584b097048b751eb7737717015aacc838fd3884bb7f17aca94220ed7d20
-  data.tar.gz: 94db3b9126927989d8a875a21a8ca096c9bb5f37eb859546295a15c891f1832f50f31e7a35971f94e57a9b5e876d0040a626fc130113269a3658110fa10eb3e5
+  metadata.gz: ec993a65a632c6fe56f4f07e6c9fde1762bd85671039dbca10b31fb93278a42a00284d5b72d119bc7335b21ab0adb58ce8b7b1bb9ab61a8b4231df8f94f6bade
+  data.tar.gz: 6e0424481fe45ed1be20b3684090cad6cbdb6a488b563482ab58728d451f3760208170f813b0a95ab34922b884ee53795d574abf8212aec4a91871beed0c03ce

data/ext/clickhouse_native/client.cpp

@@ -174,16 +174,24 @@ static VALUE value_at(const ColumnRef& col, size_t idx) {
         }
 
         case Type::Date: {
-            auto t = col->As<ColumnDate>()->At(idx);
-            return rb_time_new(t, 0);
+            // Return a Ruby Date so as_json/to_s gives "YYYY-MM-DD" instead of
+            // a full ISO8601 timestamp. CH Date is days since 1970-01-01 UTC.
+            std::time_t t = col->As<ColumnDate>()->At(idx);
+            long days = static_cast<long>(t / 86400);
+            VALUE rb_cDate = rb_const_get(rb_cObject, rb_intern("Date"));
+            return rb_funcall(rb_cDate, rb_intern("jd"), 1, LONG2NUM(days + 2440588));
         }
         case Type::Date32: {
-            auto t = col->As<ColumnDate32>()->At(idx);
-            return rb_time_new(t, 0);
+            std::time_t t = col->As<ColumnDate32>()->At(idx);
+            long days = static_cast<long>(t / 86400);
+            VALUE rb_cDate = rb_const_get(rb_cObject, rb_intern("Date"));
+            return rb_funcall(rb_cDate, rb_intern("jd"), 1, LONG2NUM(days + 2440588));
         }
         case Type::DateTime: {
             auto t = col->As<ColumnDateTime>()->At(idx);
-            return rb_time_new(t, 0);
+            VALUE rb_t = rb_time_new(t, 0);
+            rb_funcall(rb_t, rb_intern("utc"), 0);
+            return rb_t;
         }
         case Type::DateTime64: {
             auto ct = col->As<ColumnDateTime64>();
@@ -194,7 +202,9 @@ static VALUE value_at(const ColumnRef& col, size_t idx) {
             int64_t frac = ticks % denom;
             int64_t usec = (prec <= 6) ? frac * pow10_i64(6 - prec)
                                        : frac / pow10_i64(prec - 6);
-            return rb_time_new(secs, usec);
+            VALUE rb_t = rb_time_new(secs, usec);
+            rb_funcall(rb_t, rb_intern("utc"), 0);
+            return rb_t;
         }
 
         case Type::Array: {
@@ -340,6 +350,28 @@ static bool has_trailing_tz(const char* s, size_t len) {
     return false;
 }
 
+// For Date columns: normalise the value to its UTC-midnight epoch so the
+// calendar day is preserved regardless of the process's local timezone.
+// Handles Date, DateTime, Time, String ("YYYY-MM-DD" etc.), Integer epoch.
+static int64_t coerce_to_date_epoch(VALUE value) {
+    if (RB_INTEGER_TYPE_P(value)) return NUM2LL(value);
+    VALUE date_src = value;
+    if (RB_TYPE_P(value, T_STRING)) {
+        VALUE rb_cDate = rb_const_get(rb_cObject, rb_intern("Date"));
+        date_src = rb_funcall(rb_cDate, rb_intern("parse"), 1, value);
+    } else if (rb_respond_to(value, rb_intern("hour"))) {
+        // Time / DateTime: take the wall-clock date portion as UTC midnight.
+        // We intentionally ignore the TZ offset here — callers who need
+        // strict moment semantics should use DateTime64 instead.
+        date_src = value;
+    }
+    VALUE y = rb_funcall(date_src, rb_intern("year"), 0);
+    VALUE m = rb_funcall(date_src, rb_intern("month"), 0);
+    VALUE d = rb_funcall(date_src, rb_intern("day"), 0);
+    VALUE utc = rb_funcall(rb_cTime, rb_intern("utc"), 3, y, m, d);
+    return NUM2LL(rb_funcall(utc, rb_intern("to_i"), 0));
+}
+
 // Accepts a Time, a String (parsed via Time.parse), or a numeric (epoch
 // seconds). Mirrors how the HTTP gem's JSONEachRow coerced date cells.
 //
@@ -429,16 +461,8 @@ static void append_value(const ColumnRef& col, VALUE value) {
             return;
         }
 
-        case Type::Date: {
-            VALUE t = coerce_to_time(value);
-            col->As<ColumnDate>()->Append(static_cast<std::time_t>(NUM2LL(rb_funcall(t, rb_intern("to_i"), 0))));
-            return;
-        }
-        case Type::Date32: {
-            VALUE t = coerce_to_time(value);
-            col->As<ColumnDate32>()->Append(static_cast<std::time_t>(NUM2LL(rb_funcall(t, rb_intern("to_i"), 0))));
-            return;
-        }
+        case Type::Date:   col->As<ColumnDate>()->Append(static_cast<std::time_t>(coerce_to_date_epoch(value))); return;
+        case Type::Date32: col->As<ColumnDate32>()->Append(static_cast<std::time_t>(coerce_to_date_epoch(value))); return;
         case Type::DateTime: {
             VALUE t = coerce_to_time(value);
             col->As<ColumnDateTime>()->Append(static_cast<std::time_t>(NUM2LL(rb_funcall(t, rb_intern("to_i"), 0))));

data/lib/clickhouse_native/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ClickhouseNative
-  VERSION = "0.1.3"
+  VERSION = "0.2.0"
 end

data/lib/clickhouse_native.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "time"
+require "date"
 require "bigdecimal"
 require "clickhouse_native/version"
 require "clickhouse_native/errors"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: clickhouse-native
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.2.0
 platform: ruby
 authors:
 - Yuri Smirnov