ruby_snowflake_client 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 75899b8f6ba98da01620d76f6d7077ba73ae8dd7fb1177815300b969d0dbf7b5
-  data.tar.gz: 2fdbd266149c69bf17d59c83ab09418ca8f46d23707c5c3d98cc8e6800bf84e2
+  metadata.gz: 056a78a20053733ca15070ef4aec126052ad1b6eb1c3191788ffcae8acc4d80c
+  data.tar.gz: 3e33de5630c3f14f3b43e8076c672cbfa47295a28a9fc740f853a7b2dbbdb07e
 SHA512:
-  metadata.gz: 180122aebc707aeda9e7cff2fad9ab6361b21d027b4dc23961bb64319ae192338091fff3b5600cbc15a86522c39bff2b4402723c7c0dfbd80570bdea3e3f96cb
-  data.tar.gz: 65ab2555c42a044224ee1f5e010f43f7ddc9491205d271bd4459459ee5ffb3f65112bd310742c1bcab797c66e8a590e96f732032142ba3866e2e91afb5feb40b
+  metadata.gz: 215d13f6351cb22543bf1c3e356df1c98fb87e6bb2ecd2a99915a593e91316f8162e67dcce3d3e26d1135a7adf6de2538f4d2635a9bf53daf7532134e6f9a93f
+  data.tar.gz: 8c3b9d2a94d4641f96472ec923996940d1e7b99cf5f32989e0b3db431e046de23ec22009a0ce2a1002b3c5976484bea62de421d3c26de08af01de96de3bc0f09

data/.github/workflows/ci.yml

@@ -32,7 +32,7 @@ jobs:
           ### limits ssh access and adds the ssh public key for the user which triggered the workflow
           #limit-access-to-actor: true
       - name: Install gem
-        run: cd pkg && gem install --local *.gem --verbose
+        run: cd pkg && gem install --local *.gem
       - name: Run tests
         run: ruby -rruby_snowflake_client -S rspec spec/**/*_spec.rb -cfdoc
         env: # Or as an environment variable

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    ruby_snowflake_client (1.1.0)
+    ruby_snowflake_client (1.2.0)
 
 GEM
   remote: https://rubygems.org/

data/ext/client.go

@@ -0,0 +1,98 @@
+package main
+
+/*
+#include <stdlib.h>
+#include "ruby/ruby.h"
+
+void RbGcGuard(VALUE ptr);
+VALUE ReturnEnumerator(VALUE cls);
+VALUE RbNumFromDouble(double v);
+*/
+import "C"
+
+import (
+	"context"
+	"database/sql"
+	"fmt"
+	"strings"
+	"time"
+
+	sf "github.com/snowflakedb/gosnowflake"
+)
+
+type SnowflakeClient struct {
+	db *sql.DB
+}
+
+func (x SnowflakeClient) Fetch(statement C.VALUE) C.VALUE {
+	t1 := time.Now()
+
+	if LOG_LEVEL > 0 {
+		fmt.Println("statement", RbGoString(statement))
+	}
+	rows, err := x.db.QueryContext(sf.WithHigherPrecision(context.Background()), RbGoString(statement))
+	if err != nil {
+		result := C.rb_class_new_instance(0, &empty, rbSnowflakeResultClass)
+		errStr := fmt.Sprintf("Query error: '%s'", err.Error())
+		C.rb_ivar_set(result, ERROR_IDENT, RbString(errStr))
+		return result
+	}
+
+	duration := time.Now().Sub(t1).Seconds()
+	if LOG_LEVEL > 0 {
+		fmt.Printf("Query duration: %s\n", time.Now().Sub(t1))
+	}
+	if err != nil {
+		result := C.rb_class_new_instance(0, &empty, rbSnowflakeResultClass)
+		errStr := fmt.Sprintf("Query error: '%s'", err.Error())
+		C.rb_ivar_set(result, ERROR_IDENT, RbString(errStr))
+		return result
+	}
+
+	result := C.rb_class_new_instance(0, &empty, rbSnowflakeResultClass)
+	cols, _ := rows.Columns()
+	for idx, col := range cols {
+		col := col
+		cols[idx] = strings.ToLower(col)
+	}
+	rs := SnowflakeResult{rows, cols}
+	resultMap[result] = &rs
+	C.rb_ivar_set(result, RESULT_DURATION, RbNumFromDouble(C.double(duration)))
+	return result
+}
+
+//export Connect
+func Connect(self C.VALUE, account C.VALUE, warehouse C.VALUE, database C.VALUE, schema C.VALUE, user C.VALUE, password C.VALUE, role C.VALUE) {
+	// other optional parms: Application, Host, and alt auth schemes
+	cfg := &sf.Config{
+		Account:   RbGoString(account),
+		Warehouse: RbGoString(warehouse),
+		Database:  RbGoString(database),
+		Schema:    RbGoString(schema),
+		User:      RbGoString(user),
+		Password:  RbGoString(password),
+		Role:      RbGoString(role),
+		Port:      int(443),
+	}
+
+	dsn, err := sf.DSN(cfg)
+	if err != nil {
+		errStr := fmt.Sprintf("Snowflake Config Creation Error: '%s'", err.Error())
+		C.rb_ivar_set(self, ERROR_IDENT, RbString(errStr))
+	}
+
+	db, err := sql.Open("snowflake", dsn)
+	if err != nil {
+		errStr := fmt.Sprintf("Connection Error: '%s'", err.Error())
+		C.rb_ivar_set(self, ERROR_IDENT, RbString(errStr))
+	}
+	rs := SnowflakeClient{db}
+	clientRef[self] = &rs
+}
+
+//export ObjFetch
+func ObjFetch(self C.VALUE, statement C.VALUE) C.VALUE {
+	x, _ := clientRef[self]
+
+	return x.Fetch(statement)
+}

data/ext/result.go

@@ -5,12 +5,11 @@ package main
 #include "ruby/ruby.h"
 
 VALUE ReturnEnumerator(VALUE cls);
-VALUE createRbString(char* str);
-VALUE funcall0param(VALUE obj, ID id);
 */
 import "C"
 
 import (
+	"database/sql"
 	"fmt"
 	"math/big"
 	"time"
@@ -18,6 +17,11 @@ import (
 	gopointer "github.com/mattn/go-pointer"
 )
 
+type SnowflakeResult struct {
+	rows    *sql.Rows
+	columns []string
+}
+
 func wrapRbRaise(err error) {
 	fmt.Printf("[ruby-snowflake-client] Error encountered: %s\n", err.Error())
 	fmt.Printf("[ruby-snowflake-client] Will call `rb_raise`\n")

data/ext/ruby_snowflake.go

@@ -10,9 +10,6 @@ VALUE Inspect(VALUE);
 VALUE GetRows(VALUE);
 VALUE GetRowsNoEnum(VALUE);
 
-VALUE NewGoStruct(VALUE klass, char* reason, void *p);
-VALUE GoRetEnum(VALUE,int,VALUE);
-void* GetGoStruct(VALUE obj);
 void RbGcGuard(VALUE ptr);
 VALUE ReturnEnumerator(VALUE cls);
 VALUE RbNumFromDouble(double v);
@@ -20,25 +17,9 @@ VALUE RbNumFromDouble(double v);
 import "C"
 
 import (
-	"context"
-	"database/sql"
 	"fmt"
-	"strings"
-	"time"
-
-	sf "github.com/snowflakedb/gosnowflake"
 )
 
-type SnowflakeResult struct {
-	rows *sql.Rows
-	//keptHash C.VALUE
-	columns []string
-	//cols     []C.VALUE
-}
-type SnowflakeClient struct {
-	db *sql.DB
-}
-
 var rbSnowflakeClientClass C.VALUE
 var rbSnowflakeResultClass C.VALUE
 var rbSnowflakeModule C.VALUE
@@ -54,79 +35,6 @@ var clientRef = make(map[C.VALUE]*SnowflakeClient)
 var LOG_LEVEL = 0
 var empty C.VALUE = C.Qnil
 
-//export Connect
-func Connect(self C.VALUE, account C.VALUE, warehouse C.VALUE, database C.VALUE, schema C.VALUE, user C.VALUE, password C.VALUE, role C.VALUE) {
-	// other optional parms: Application, Host, and alt auth schemes
-	cfg := &sf.Config{
-		Account:   RbGoString(account),
-		Warehouse: RbGoString(warehouse),
-		Database:  RbGoString(database),
-		Schema:    RbGoString(schema),
-		User:      RbGoString(user),
-		Password:  RbGoString(password),
-		Role:      RbGoString(role),
-		Port:      int(443),
-	}
-
-	dsn, err := sf.DSN(cfg)
-	if err != nil {
-		errStr := fmt.Sprintf("Snowflake Config Creation Error: '%s'", err.Error())
-		C.rb_ivar_set(self, ERROR_IDENT, RbString(errStr))
-	}
-
-	db, err := sql.Open("snowflake", dsn)
-	if err != nil {
-		errStr := fmt.Sprintf("Connection Error: '%s'", err.Error())
-		C.rb_ivar_set(self, ERROR_IDENT, RbString(errStr))
-	}
-	rs := SnowflakeClient{db}
-	clientRef[self] = &rs
-}
-
-func (x SnowflakeClient) Fetch(statement C.VALUE) C.VALUE {
-	t1 := time.Now()
-
-	if LOG_LEVEL > 0 {
-		fmt.Println("statement", RbGoString(statement))
-	}
-	rows, err := x.db.QueryContext(sf.WithHigherPrecision(context.Background()), RbGoString(statement))
-	if err != nil {
-		result := C.rb_class_new_instance(0, &empty, rbSnowflakeResultClass)
-		errStr := fmt.Sprintf("Query error: '%s'", err.Error())
-		C.rb_ivar_set(result, ERROR_IDENT, RbString(errStr))
-		return result
-	}
-
-	duration := time.Now().Sub(t1).Seconds()
-	if LOG_LEVEL > 0 {
-		fmt.Printf("Query duration: %s\n", time.Now().Sub(t1))
-	}
-	if err != nil {
-		result := C.rb_class_new_instance(0, &empty, rbSnowflakeResultClass)
-		errStr := fmt.Sprintf("Query error: '%s'", err.Error())
-		C.rb_ivar_set(result, ERROR_IDENT, RbString(errStr))
-		return result
-	}
-
-	result := C.rb_class_new_instance(0, &empty, rbSnowflakeResultClass)
-	cols, _ := rows.Columns()
-	for idx, col := range cols {
-		col := col
-		cols[idx] = strings.ToLower(col)
-	}
-	rs := SnowflakeResult{rows, cols}
-	resultMap[result] = &rs
-	C.rb_ivar_set(result, RESULT_DURATION, RbNumFromDouble(C.double(duration)))
-	return result
-}
-
-//export ObjFetch
-func ObjFetch(self C.VALUE, statement C.VALUE) C.VALUE {
-	x, _ := clientRef[self]
-
-	return x.Fetch(statement)
-}
-
 //export Inspect
 func Inspect(self C.VALUE) C.VALUE {
 	x := clientRef[self]

data/ext/wrapper.go

@@ -24,57 +24,14 @@ VALUE RbNumFromLong(long v) {
 	return LONG2NUM(v);
 }
 
-void goobj_retain(void *, char*);
-void goobj_free(void *);
-void goobj_log(void *);
-void goobj_mark(void *);
-void goobj_compact(void *);
-
-static const rb_data_type_t go_type = {
-    "GoStruct",
-    {
-			goobj_mark,
-			goobj_free,
-			NULL,
-			(goobj_compact),
-		},
-    0, 0, RUBY_TYPED_FREE_IMMEDIATELY
-};
-
-VALUE
-NewGoStruct(VALUE klass, char* reason, void *p)
-{
-    goobj_retain(p, reason);
-    return TypedData_Wrap_Struct(klass, &go_type, p);
-}
-
 VALUE ReturnEnumerator(VALUE cls) {
 	RETURN_ENUMERATOR(cls, 0, NULL);
 	return Qnil;
 }
 
-void *
-GetGoStruct(VALUE obj)
-{
-    void *val;
-    return TypedData_Get_Struct(obj, void *, &go_type, val);
-}
-
 void RbGcGuard(VALUE ptr) {
 	RB_GC_GUARD(ptr);
 }
-
-VALUE createRbString(char* str) {
-	volatile VALUE rbStr;
-	rbStr = rb_tainted_str_new_cstr(str);
-	return rbStr;
-}
-
-VALUE funcall0param(VALUE obj, ID id) {
-	RB_GC_GUARD(obj);
-	return rb_funcall(obj, id, 0);
-}
-
 */
 import "C"
 import (
@@ -90,10 +47,6 @@ func RbNumFromDouble(v C.double) C.VALUE {
 	return C.RbNumFromDouble(v)
 }
 
-func GetGoStruct(obj C.VALUE) unsafe.Pointer {
-	return C.GetGoStruct(obj)
-}
-
 func returnEnum(cls C.VALUE) C.VALUE {
 	return C.ReturnEnumerator(cls)
 }

data/lib/ruby_snowflake_client/version.rb

@@ -1,3 +1,3 @@
 module RubySnowflakeClient
-  VERSION = '1.1.0'
+  VERSION = '1.2.0'
 end

data/lib/ruby_snowflake_client.rb

@@ -5,10 +5,12 @@ module Snowflake
   LOG_LEVEL = 0
 
   class Error < StandardError
-    attr_reader :details
+    # This will get pulled through to Sentry, see:
+    # https://github.com/getsentry/sentry-ruby/blob/11ecd254c0d2cae2b327f0348074e849095aa32d/sentry-ruby/lib/sentry/error_event.rb#L31-L33
+    attr_reader :sentry_context
 
     def initialize(details)
-      @details = details
+      @sentry_context = details
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby_snowflake_client
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Rinsed
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-06-09 00:00:00.000000000 Z
+date: 2023-06-26 00:00:00.000000000 Z
 dependencies: []
 description: |
   Using the `Go` library for Snowflake to query and creating native Ruby objects,
@@ -29,7 +29,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
-- ext/c-decl.go
+- ext/client.go
 - ext/extconf.rb
 - ext/go.mod
 - ext/go.sum

data/ext/c-decl.go

@@ -1,54 +0,0 @@
-package main
-
-/*
-#include <stdlib.h>
-#include "ruby/ruby.h"
-*/
-import "C"
-
-import (
-	"fmt"
-	"unsafe"
-)
-
-var marked = make(map[unsafe.Pointer]int)
-
-//export goobj_mark
-func goobj_mark(obj unsafe.Pointer) {
-	if LOG_LEVEL > 0 {
-		marked[obj] = marked[obj] + 1
-		fmt.Printf("MARK log obj %v; counter: %d; total number of MARKED objects: %d\n", obj, marked[obj], len(marked))
-	}
-}
-
-//export goobj_log
-func goobj_log(obj unsafe.Pointer) {
-	if LOG_LEVEL > 0 {
-		fmt.Println("log obj", obj)
-	}
-}
-
-//export goobj_retain
-func goobj_retain(obj unsafe.Pointer, x *C.char) {
-	if LOG_LEVEL > 0 {
-		fmt.Printf("retain obj [%v] %v - currently keeping %d\n", C.GoString(x), obj, len(objects))
-	}
-	objects[obj] = true
-	marked[obj] = 0
-}
-
-//export goobj_free
-func goobj_free(obj unsafe.Pointer) {
-	if LOG_LEVEL > 0 {
-		fmt.Printf("CALLED GOOBJ FREE %v - CURRENTLY %d objects left\n", obj, len(objects))
-	}
-	delete(objects, obj)
-	delete(marked, obj)
-}
-
-//export goobj_compact
-func goobj_compact(obj unsafe.Pointer) {
-	if LOG_LEVEL > 0 {
-		fmt.Printf("CALLED GOOBJ COMPACT %v", obj)
-	}
-}