search

dosco / graphjin

GraphJin - Build NodeJS / GO APIs in 5 minutes not weeks
https://graphjin.com
Apache License 2.0
2.91k stars 177 forks source link

Telem bump #506

Closed AbdulDridi closed 4 months ago

AbdulDridi commented 4 months ago

Fix for the "Conflicting schema URL" error I was seeing locally

Screenshot 2024-05-08 at 19 36 52

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
graphjin ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2024 9:33pm
dosco commented 4 months ago

this is a little more complex then bumping that number since the apis of that library have changed. i'm in the process of a new released with all these dependencies upgraded to the latest 3.0.25 should be out in about 15 minutes.

AbdulDridi commented 4 months ago

Ah ok, I'll close this PR in that case! Thanks for giving it a look.

An additional question on my part, perhaps better reserved for a discord discussion? - From my tinkering, it seems open telemetry is currently configured for the api layer, but no traces are generated for the DB (The information i'm interested in is the exact populated SQL queries being executed for debugging/tuning)

Would I have to setup https://pkg.golang.ir/github.com/exaring/otelpgx to support this?

dosco commented 4 months ago

if you think its useful we should add it in.