duckdb 0.8.1.2 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: def2e5adaab85bedca1363e7c527640eeb7f2cc2938c4bac219b026d8d79e895
-  data.tar.gz: 2f058e71abd48266432f7486bed43a67fa8ef5e8955ccc150c27a64e023fb7d5
+  metadata.gz: f7738df010c1fb1aab32b61077ff9409566cfaddecfef3b27aa6e7ee7a8651f2
+  data.tar.gz: 6357f43da56b7c4fbea72ec7b3c549f598f3a91b88c1c91ab5320a01606f3589
 SHA512:
-  metadata.gz: 388302d5e187b6f8f85877c0cfd2613e78047ba78484067e99e8fe1a6cee16ca61a59918ae4d9090b4b9029e8594f0c4f63e2d9855f3f3a931979b51430ecd8f
-  data.tar.gz: 27418556352f19324ebc8e90cae28972f05965cd2394dccd92836ccf1dd044604f2e1045ebdb7f05b28afeec493de07fa1d060dde3c3367a60fbcf4fee30038f
+  metadata.gz: e417badc1b301fdea9f3f4b27648e7e1c7d8e125a0a1323bb69e93a992467eca0a57a320b0bb14f7fda45bf8d9d3706b59eb586a71e3c9f77bd2ec18c88b8b3a
+  data.tar.gz: b1486d0d525fa93a8485504eac95015a07ac064a4dfb15bb0689aa7be4cb41edcf03436324e822bda26a8c3c6e8a9e2a690efcd49f7180f675777f75284df5e8

data/.github/FUNDING.yml

@@ -0,0 +1,3 @@
+# These are supported funding model platforms
+
+github: suketa

data/.github/workflows/test_on_macos.yml

@@ -16,7 +16,7 @@ jobs:
     strategy:
       matrix:
         ruby: ['2.7.8', '3.0.6', '3.1.4', '3.2.2', '3.3.0-preview1', 'head']
-        duckdb: ['0.7.1', '0.8.1']
+        duckdb: ['0.9.0', '0.8.1']
 
     steps:
     - uses: actions/checkout@v3

data/.github/workflows/test_on_ubuntu.yml

@@ -16,7 +16,7 @@ jobs:
     strategy:
       matrix:
         ruby: ['2.7.8', '3.0.6', '3.1.4', '3.2.2', '3.3.0-preview1', 'head']
-        duckdb: ['0.7.1', '0.8.1']
+        duckdb: ['0.9.0', '0.8.1']
 
     steps:
     - uses: actions/checkout@v3

data/.github/workflows/test_on_windows.yml

@@ -16,7 +16,7 @@ jobs:
     strategy:
       matrix:
         ruby: ['2.7.8', '3.0.6', '3.1.4', '3.2.2', 'ucrt', 'mingw', 'mswin', 'head']
-        duckdb: ['0.7.1', '0.8.1']
+        duckdb: ['0.9.0', '0.8.1']
 
     steps:
     - uses: actions/checkout@v3

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # ChangeLog
 
+# 0.9.0
+- bump duckdb to 0.9.0.
+
+## Breaking Change
+- deprecation warning when DuckDB::Result.each calling with `DuckDB::Result.use_chunk_each` is false.
+  The `each` behavior will be same as `DuckDB::Result.chunk_each` in the future.
+  set `DuckDB::Result.use_chunk_each = true` to suppress the warning.
+- DuckDB::Result#chunck_each returns DuckDB::Interval class when the column type is INTERVAL.
+
+# 0.8.1.3
+- Fix BigDecimal conversion.
+
 # 0.8.1.2
 - Fix BigDecimal conversion when the value is 0.
   Thanks to shreeve.

data/CONTRIBUTION.md

@@ -8,8 +8,8 @@
 2. Build and access to docker container
 
 ```
-docker-compose build ubuntu
-docker-compose run --rm ubuntu bash
+docker compose build ubuntu
+docker compose run --rm ubuntu bash
 ```
 
 In case you want custom ruby or duckdb versions, use `--build-arg` options

data/Dockerfile

@@ -1,7 +1,7 @@
 ARG RUBY_VERSION=3.2.2
 FROM ruby:${RUBY_VERSION}
 
-ARG DUCKDB_VERSION=0.8.0
+ARG DUCKDB_VERSION=0.9.0
 
 RUN apt update -qq && \
     apt install -y build-essential curl git wget

data/Gemfile.lock

@@ -1,24 +1,24 @@
 PATH
   remote: .
   specs:
-    duckdb (0.8.1.2)
+    duckdb (0.9.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
     benchmark-ips (2.12.0)
-    mini_portile2 (2.8.2)
-    minitest (5.18.1)
-    nokogiri (1.15.3)
+    mini_portile2 (2.8.4)
+    minitest (5.20.0)
+    nokogiri (1.15.4)
       mini_portile2 (~> 2.8.2)
       racc (~> 1.4)
-    nokogiri (1.15.3-x86_64-linux)
+    nokogiri (1.15.4-x86_64-linux)
       racc (~> 1.4)
     racc (1.7.1)
     rake (13.0.6)
-    rake-compiler (1.2.3)
+    rake-compiler (1.2.5)
       rake
-    ruby_memcheck (1.3.2)
+    ruby_memcheck (2.2.0)
       nokogiri
     stackprof (0.2.25)
 

data/benchmark/converter_hugeint_ips.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+require 'bundler/setup'
+require 'duckdb'
+require 'benchmark/ips'
+
+Benchmark.ips do |x|
+  x.report('hugeint_convert') { DuckDB::Converter._to_hugeint_from_vector(123_456_789, 123_456_789) }
+end
+
+__END__
+
+## before
+```
+✦ ❯ ruby benchmark/converter_hugeint_ips.rb
+Warming up --------------------------------------
+       hugeint_convert   318.524k i/100ms
+Calculating -------------------------------------
+       hugeint_convert      3.940M (± 0.7%) i/s -     19.748M in   5.012440s
+```
+
+## after (use bit shift)
+✦ ❯ ruby benchmark/converter_hugeint_ips.rb
+Warming up --------------------------------------
+       hugeint_convert   347.419k i/100ms
+Calculating -------------------------------------
+       hugeint_convert      4.171M (± 0.3%) i/s -     21.193M in   5.081131s

data/ext/duckdb/duckdb.c

@@ -4,8 +4,16 @@ VALUE mDuckDB;
 
 static VALUE duckdb_s_library_version(VALUE self);
 
+/*
+ * call-seq:
+ *   DuckDB.library_version -> String
+ *
+ * Returns the version of the DuckDB library.
+ *
+ *   DuckDB.library_version # => "0.2.0"
+ */
 static VALUE duckdb_s_library_version(VALUE self) {
-  return rb_str_new2(duckdb_library_version());
+    return rb_str_new2(duckdb_library_version());
 }
 
 void

data/ext/duckdb/extconf.rb

@@ -28,4 +28,7 @@ have_func('duckdb_extract_statements', 'duckdb.h')
 # check duckdb >= 0.8.0
 have_func('duckdb_string_is_inlined', 'duckdb.h')
 
+# check duckdb >= 0.9.0
+have_func('duckdb_bind_parameter_index', 'duckdb.h')
+
 create_makefile('duckdb/duckdb_native')

data/ext/duckdb/ruby-duckdb.h

@@ -12,6 +12,10 @@
 #define HAVE_DUCKDB_H_GE_V080 1
 #endif
 
+#ifdef HAVE_DUCKDB_BIND_PARAMETER_INDEX
+#define HAVE_DUCKDB_H_GE_V090 1
+#endif
+
 #include "./error.h"
 #include "./database.h"
 #include "./connection.h"

data/lib/duckdb/appender.rb

@@ -150,13 +150,8 @@ module DuckDB
     #     .flush
     #
     def append_interval(value)
-      raise ArgumentError, "Argument `#{value}` must be a string." unless value.is_a?(String)
-
-      hash = iso8601_interval_to_hash(value)
-
-      months, days, micros = hash_to__append_interval_args(hash)
-
-      _append_interval(months, days, micros)
+      value = Interval.to_interval(value)
+      _append_interval(value.interval_months, value.interval_days, value.interval_micros)
     end
 
     #
@@ -197,6 +192,8 @@ module DuckDB
         append_timestamp(value)
       when Date
         append_date(value)
+      when DuckDB::Interval
+        append_interval(value)
       else
         raise(DuckDB::Error, "not supported type #{value} (#{value.class})")
       end

data/lib/duckdb/converter.rb

@@ -1,10 +1,12 @@
 # frozen_string_literal: true
 
 require 'date'
+require_relative 'interval'
 
 module DuckDB
   module Converter
-    HALF_HUGEINT = 1 << 64
+    HALF_HUGEINT_BIT = 64
+    HALF_HUGEINT = 1 << HALF_HUGEINT_BIT
     FLIP_HUGEINT = 1 << 63
 
     module_function
@@ -18,25 +20,18 @@ module DuckDB
     end
 
     def _to_hugeint_from_vector(lower, upper)
-      (upper * HALF_HUGEINT) + lower
+      (upper << HALF_HUGEINT_BIT) + lower
     end
 
     def _to_decimal_from_vector(_width, scale, lower, upper)
       v = _to_hugeint_from_vector(lower, upper).to_s
-      v[-scale, 0] = '.' unless v == '0'
+      v = v.rjust(scale + 1, '0') if v.length < scale
+      v[-scale, 0] = '.'
       BigDecimal(v)
     end
 
     def _to_interval_from_vector(months, days, micros)
-      hash = { year: 0, month: 0, day: 0, hour: 0, min: 0, sec: 0, usec: 0 }
-      hash[:year] = months / 12
-      hash[:month] = months % 12
-      hash[:day] = days
-      hash[:hour] = micros / 3_600_000_000
-      hash[:min] = (micros % 3_600_000_000) / 60_000_000
-      hash[:sec] = (micros % 60_000_000) / 1_000_000
-      hash[:usec] = micros % 1_000_000
-      hash
+      Interval.new(interval_months: months, interval_days: days, interval_micros: micros)
     end
 
     def _to_uuid_from_vector(lower, upper)
@@ -52,59 +47,12 @@ module DuckDB
     def integer_to_hugeint(value)
       case value
       when Integer
-        upper = value / HALF_HUGEINT
-        lower = value - upper * HALF_HUGEINT
+        upper = value >> HALF_HUGEINT_BIT
+        lower = value - (upper << HALF_HUGEINT_BIT)
         [lower, upper]
       else
-        raise(ArgumentError, "2nd argument `#{value}` must be Integer.")
+        raise(ArgumentError, "The argument `#{value}` must be Integer.")
       end
     end
-
-    def iso8601_interval_to_hash(value)
-      digit = ''
-      time = false
-      hash = {}
-      hash.default = 0
-
-      value.each_char do |c|
-        if '-0123456789.'.include?(c)
-          digit += c
-        elsif c == 'T'
-          time = true
-          digit = ''
-        elsif c == 'M'
-          m_interval_to_hash(hash, digit, time)
-          digit = ''
-        elsif c == 'S'
-          s_interval_to_hash(hash, digit)
-          digit = ''
-        elsif 'YDH'.include?(c)
-          hash[c] = digit.to_i
-          digit = ''
-        elsif c != 'P'
-          raise ArgumentError, "The argument `#{value}` can't be parse."
-        end
-      end
-      hash
-    end
-
-    def m_interval_to_hash(hash, digit, time)
-      key = time ? 'TM' : 'M'
-      hash[key] = digit.to_i
-    end
-
-    def s_interval_to_hash(hash, digit)
-      sec, msec = digit.split('.')
-      hash['S'] = sec.to_i
-      hash['MS'] = "#{msec}000000"[0, 6].to_i
-      hash['MS'] *= -1 if hash['S'].negative?
-    end
-
-    def hash_to__append_interval_args(hash)
-      months = hash['Y'] * 12 + hash['M']
-      days = hash['D']
-      micros = (hash['H'] * 3600 + hash['TM'] * 60 + hash['S']) * 1_000_000 + hash['MS']
-      [months, days, micros]
-    end
   end
 end

data/lib/duckdb/interval.rb

@@ -0,0 +1,161 @@
+# frozen_string_literal: true
+
+module DuckDB
+  # Interval class represents DuckDB's interval type value.
+  #
+  # The usage is as follows:
+  #
+  #   require 'duckdb'
+  #
+  #   interval = DuckDB::Interval.new(interval_months: 14, interval_days: 3, interval_micros: 14706123456)
+  #   # or
+  #   # interval = DuckDB::Interval.mk_interval(year: 1, month: 2, day: 3, hour: 4, min: 5, sec: 6, usec: 123456)
+  #   # or
+  #   # interval = DuckDB::Interval.iso8601_parse('P1Y2M3DT4H5M6.123456S')
+  #
+  #   db = DuckDB::Database.open # database in memory
+  #   con = db.connect
+  #
+  #   con.execute('CREATE TABLE users (value INTERVAL)')
+  #
+  #   require 'duckdb'
+  #   db = DuckDB::Database.open
+  #   con = db.connect
+  #   con.query('CREATE TABLE intervals (interval_value INTERVAL)')
+  #   appender = con.appender('intervals')
+  #   appender
+  #     .begin_row
+  #     .append_interval(interval)
+  #     .end_row
+  #     .flush
+  class Interval
+    ISO8601_REGEXP = Regexp.compile(
+      '(?<negativ>-{0,1})P
+      (?<year>-{0,1}\d+Y){0,1}
+      (?<month>-{0,1}\d+M){0,1}
+      (?<day>-{0,1}\d+D){0,1}
+      T{0,1}
+      (?<hour>-{0,1}\d+H){0,1}
+      (?<min>-{0,1}\d+M){0,1}
+      ((?<sec>-{0,1}\d+)\.{0,1}(?<usec>\d*)S){0,1}',
+      Regexp::EXTENDED
+    )
+
+    class << self
+      # parses the ISO8601 format string and return the Interval object.
+      #
+      #   DuckDB::Interval.iso8601_parse('P1Y2M3DT4H5M6.123456S')
+      #   => #<DuckDB::Interval:0x00007f9b9c0b3b60 @interval_months=14, @interval_days=3, @interval_micros=14706123456>
+      def iso8601_parse(value)
+        m = ISO8601_REGEXP.match(value)
+
+        raise ArgumentError, "The argument `#{value}` can't be parse." if m.nil?
+
+        year, month, day, hour, min, sec, usec = matched_to_i(m)
+
+        mk_interval(year: year, month: month, day: day, hour: hour, min: min, sec: sec, usec: usec)
+      end
+
+      # creates the Interval object.
+      #
+      #   DuckDB::Interval.mk_interval(year: 1, month: 2, day: 3, hour: 4, min: 5, sec: 6, usec: 123456)
+      #   => #<DuckDB::Interval:0x00007f9b9c0b3b60 @interval_months=14, @interval_days=3, @interval_micros=14706123456>
+      def mk_interval(year: 0, month: 0, day: 0, hour: 0, min: 0, sec: 0, usec: 0)
+        Interval.new(
+          interval_months: (year * 12) + month,
+          interval_days: day,
+          interval_micros: (((hour * 3600) + (min * 60) + sec) * 1_000_000) + usec
+        )
+      end
+
+      # Convert the value to the Interval object.
+      # The value can be String or Interval object.
+      # If the value is String, it is parsed as ISO8601 format.
+      # If the value is Interval object, it is returned as is.
+      # Otherwise, ArgumentError is raised.
+      #
+      #   DuckDB::Interval.to_interval('P1Y2M3DT4H5M6.123456S')
+      #   => #<DuckDB::Interval:0x00007f9b9c0b3b60 @interval_months=14, @interval_days=3, @interval_micros=14706123456>
+      #
+      #   interval = DuckDB::Interval.to_interval('P1Y2M3DT4H5M6.123456S')
+      #   DuckDB::Interval.to_interval(interval)
+      #   => #<DuckDB::Interval:0x00007f9b9c0b3b60 @interval_months=14, @interval_days=3, @interval_micros=14706123456>
+      def to_interval(value)
+        case value
+        when String
+          iso8601_parse(value)
+        when Interval
+          value
+        else
+          raise ArgumentError, "The argument `#{value}` can't be parse."
+        end
+      end
+
+      private
+
+      def matched_to_i(matched)
+        sign = to_sign(matched)
+        sec = to_sec(matched)
+        usec = to_usec(matched)
+        usec *= -1 if sec.negative?
+        value = [
+          to_year(matched), to_month(matched), to_day(matched), to_hour(matched), to_min(matched), sec, usec
+        ]
+        sign.positive? ? value : value.map { |v| v * sign }
+      end
+
+      def to_sign(matched)
+        matched[:negativ] == '-' ? -1 : 1
+      end
+
+      def to_year(matched)
+        matched[:year].to_i
+      end
+
+      def to_month(matched)
+        matched[:month].to_i
+      end
+
+      def to_day(matched)
+        matched[:day].to_i
+      end
+
+      def to_hour(matched)
+        matched[:hour].to_i
+      end
+
+      def to_min(matched)
+        matched[:min].to_i
+      end
+
+      def to_sec(matched)
+        matched[:sec].to_i
+      end
+
+      def to_usec(matched)
+        matched[:usec].to_s.ljust(6, '0')[0, 6].to_i
+      end
+    end
+
+    attr_reader :interval_months, :interval_days, :interval_micros
+
+    # creates the Interval object.
+    # The arguments are the number of months, days, and microseconds.
+    # The default value is 0.
+    #
+    #   DuckDB::Interval.new(interval_months: 1, interval_days: 2, interval_micros: 3)
+    #   => #<DuckDB::Interval:0x00007f9b9c0b3b60 @interval_months=1, @interval_days=2, @interval_micros=3>
+    def initialize(interval_months: 0, interval_days: 0, interval_micros: 0)
+      @interval_months = interval_months
+      @interval_days = interval_days
+      @interval_micros = interval_micros
+    end
+
+    def ==(other)
+      other.is_a?(Interval) &&
+        @interval_months == other.interval_months &&
+        @interval_days == other.interval_days &&
+        @interval_micros == other.interval_micros
+    end
+  end
+end

data/lib/duckdb/prepared_statement.rb

@@ -152,13 +152,8 @@ module DuckDB
     #   stmt = PreparedStatement.new(con, sql)
     #   stmt.bind(1, 'P1Y2D')
     def bind_interval(i, value)
-      raise ArgumentError, "Argument `#{value}` must be a string." unless value.is_a?(String)
-
-      hash = iso8601_interval_to_hash(value)
-
-      months, days, micros = hash_to__append_interval_args(hash)
-
-      _bind_interval(i, months, days, micros)
+      value = Interval.to_interval(value)
+      _bind_interval(i, value.interval_months, value.interval_days, value.interval_micros)
     end
 
     # binds i-th parameter with SQL prepared statement.

data/lib/duckdb/result.rb

@@ -58,6 +58,7 @@ module DuckDB
 
         chunk_each { |row| yield row }
       else
+        warn('this `each` behavior will be deprecated in the future. set `Result.use_chunk_each = true` to use new `each` behavior.')
         return to_enum { row_size } unless block_given?
 
         row_count.times do |row_index|
@@ -94,7 +95,7 @@ module DuckDB
 
     def _to_hugeint_internal(row, col)
       lower, upper = __to_hugeint_internal(row, col)
-      upper * Converter::HALF_HUGEINT + lower
+      Converter._to_hugeint_from_vector(lower, upper)
     end
 
     def _to_decimal(row, col)
@@ -102,10 +103,8 @@ module DuckDB
     end
 
     def _to_decimal_internal(row, col)
-      lower, upper, _width, scale = __to_decimal_internal(row, col)
-      v = (upper * Converter::HALF_HUGEINT + lower).to_s
-      v[-scale, 0] = '.' unless v == '0'
-      BigDecimal(v)
+      lower, upper, width, scale = __to_decimal_internal(row, col)
+      Converter._to_decimal_from_vector(width, scale, lower, upper)
     end
   end
 end

data/lib/duckdb/version.rb

@@ -3,5 +3,5 @@
 module DuckDB
   # The version string of ruby-duckdb.
   # Currently, ruby-duckdb is NOT semantic versioning.
-  VERSION = '0.8.1.2'
+  VERSION = '0.9.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: duckdb
 version: !ruby/object:Gem::Version
-  version: 0.8.1.2
+  version: 0.9.0
 platform: ruby
 authors:
 - Masaki Suketa
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-07-15 00:00:00.000000000 Z
+date: 2023-09-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -75,6 +75,7 @@ extensions:
 - ext/duckdb/extconf.rb
 extra_rdoc_files: []
 files:
+- ".github/FUNDING.yml"
 - ".github/workflows/test_on_macos.yml"
 - ".github/workflows/test_on_ubuntu.yml"
 - ".github/workflows/test_on_windows.yml"
@@ -87,6 +88,7 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- benchmark/converter_hugeint_ips.rb
 - benchmark/get_converter_module_ips.rb
 - benchmark/to_bigdecimal_ips.rb
 - benchmark/to_hugeint_ips.rb
@@ -128,6 +130,7 @@ files:
 - lib/duckdb/connection.rb
 - lib/duckdb/converter.rb
 - lib/duckdb/database.rb
+- lib/duckdb/interval.rb
 - lib/duckdb/library_version.rb
 - lib/duckdb/prepared_statement.rb
 - lib/duckdb/result.rb