snowflake_rb 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 14a0e8c270eb195ebf5d0fbcc60132e81814bee410fb98e5483c68f4f50a3aee
-  data.tar.gz: b67d8ce0fbd1fda9d5fa61f1f3c3c8fef35e644ab6fb664f16cd51b1891db348
+  metadata.gz: 0a94c064ba5cf498085fca7c2138d08b352f83b3a94a2a1f7465fb3f9de13c50
+  data.tar.gz: b5772f5dfa28f418a50dbdd60d579fdab0e0eb43d34c8e88e9e20d88fc9be923
 SHA512:
-  metadata.gz: 37b3214d133e4ffd8254416d17f1b55875a817ec7db87acf533e135334871ee03baf82e60457e335badede907c5cf84ca555a24ed6ca2e94e1f07c7816c8f2c8
-  data.tar.gz: b0d8f9b7c5ef46288cb3f797061d2846f93915c756253ea692ec3f42089fd59a1089083469c5bc2028a9715b30d50511e2e49f4dcbc964558455f3e64170e620
+  metadata.gz: 51abadd71833419043c7d569c22e9a6be05191db4dcfa3a014bf2b8ccf480bb43ec6c8637e518d8def995bd9a334afb6263219cc6c5c7e0e24134d4677a32052
+  data.tar.gz: b1cf22485fc54d7d47f26e6a5e0980f089024325a84a8e54cad86c829e7f7978b047482112b818d6dd253c0007e3e80a888f6b73c916db3874eb09b7e36a8ef3

data/lib/snowflake_rb.rb

@@ -2,9 +2,6 @@ require 'ffi'
 require 'snowflake_rb/extension'
 require 'snowflake_rb/configuration'
 
-# Note: this library is not thread safe as it caches the last error
-# The call pattern expectation is to call last_error after any call which may have gotten an error. If last_error is
-# `nil`, there was no error.
 module Snowflake
   module LibC
     extend FFI::Library
@@ -19,7 +16,6 @@ module Snowflake
     POINTER_SIZE = FFI.type_size(:pointer)
 
     ffi_lib(File.expand_path(SnowflakeRB::Extension.env_specific_extension_path, __dir__))
-    attach_function(:last_error, 'LastError', [], :strptr)
     # ugh, `port` in gosnowflake is just :int; however, ruby - ffi -> go is passing 32bit int if I just decl :int.
     attach_function(:connect, 'Connect', %i[string string string string string string string int64], :pointer)
     attach_function(:close, 'Close', [:pointer], :void)
@@ -33,13 +29,6 @@ module Snowflake
   module Client
     module_function
 
-    # @return String last error or nil. May be end of file which is not really an error
-    def last_error
-      error, cptr = ::Snowflake::Binding.last_error
-      LibC.free(cptr) if error
-      error
-    end
-
     # @param account[String] should include everything in the db url ahead of 'snowflakecomputing.com'
     # @param port[Integer]
     # @return query_object[Pointer] a pointer to use for subsequent calls not inspectable nor viewable by Ruby
@@ -88,12 +77,12 @@ module Snowflake
       return to_enum(__method__, db_pointer, sql) unless block_given?
 
       query_pointer = fetch(db_pointer, sql)
-      return last_error if query_pointer.nil? || query_pointer == FFI::Pointer::NULL
+      return 'query_pointer is missing' if query_pointer.nil? || query_pointer == FFI::Pointer::NULL
 
       field_count ||= column_count(query_pointer)
       loop do
         row = get_next_row(query_pointer, field_count)
-        return last_error unless row
+        return 'row not present' unless row
 
         yield row
       end
@@ -103,7 +92,6 @@ module Snowflake
     # @param db_pointer[Pointer] the pointer which `connect` returned.
     # @param query[String] a select query to run.
     # @return query_object[Pointer] a pointer to use for subsequent calls not inspectable nor viewable by Ruby; however,
-    #   if it's `nil`, check `last_error`
     def fetch(db_pointer, query)
       ::Snowflake::Binding.fetch(db_pointer, query)
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: snowflake_rb
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Darren Cheng
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-04-13 00:00:00.000000000 Z
+date: 2023-05-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi