search

wyhaines / opentelemetry-instrumentation.cr

Bundled integrations for opentelemetry-api.cr (https://github.com/wyhaines/opentelemetry-api.cr).
Apache License 2.0
16 stars 4 forks source link

Error: no overload matches for CrystalDB instrumentation with UUID #23

Closed jwoertink closed 2 years ago

jwoertink commented 2 years ago

My app uses UUID for primary keys in the DB. I'm guessing that's why the error :thinking: 

web          | Showing last frame. Use --error-trace for full trace.
web          | 
web          | In lib/opentelemetry-instrumentation/src/opentelemetry/instrumentation/crystal/db.cr:95:40
web          | 
web          |  95 | filter_result = filter.call(arg)
web          |                              ^---
web          | Error: no overload matches 'Proc((Array(Bool) | Array(Float32) | Array(Float64) | Array(Int32) | Array(Int64) | Array(Nil) | Array(Slice(UInt8)) | Array(String) | Array(Time) | Bool | Float32 | Float64 | Int32 | Int64 | Slice(UInt8) | String | Time | Nil), String)#call' with type UUID
web          | 
web          | Overloads are:
web          |  - Proc(*T, R)#call(*args : *T)
web          | 
web          | ---
wyhaines commented 2 years ago

This issue has been resolved with a small change in release 0.3.5.