Cloudflare Developers•3y ago

Writing join queries with D1 is near impossible with this issue Alex pointed out

Ddivby0 Writing join queries with D1 is near impossible with this issue Alex pointed out

Matt Silverlock•5/15/23, 9:03 AM

Use stmt.raw()stmt.raw()

https://developers.cloudflare.com/d1/platform/client-api/#await-stmtraw

D1 client API · Cloudflare D1 docs

As part of our Client API, both static and prepared statements are supported. Best practice is to use prepared statements which are precompiled …

MMatt Silverlock Use `stmt.raw()` - https://developers.cloudflare.com/d1/platform/client-api/#awa...

divby0OP•5/15/23, 9:05 AM

@alexblokh didn't tag before - are you guys using this in drizzle or is it something that could work in the near future?

divby0OP•5/15/23, 9:37 AM

I guess this could potentially also break more than just the drizzle orm integration, we just haven't found out

AAndrii Sherman I cannot identify in which version it stopped working, but I can confirm that `....

Matt Silverlock•5/15/23, 10:05 AM

Looking into it.

AAndrii Sherman I cannot identify in which version it stopped working, but I can confirm that `....

Matt Silverlock•5/15/23, 10:13 AM

Also: are you positive it ever worked? @rozenmd isn’t confident it did. Not yet at my computer but about to take a look at the D1 API internally…

Max (@rozenmd)•5/15/23, 10:14 AM

mainly because that raw implementation hasn't changed in 12ish months

MMax (@rozenmd)mainly because that raw implementation hasn't changed in 12ish months

Matt Silverlock•5/15/23, 10:15 AM

Heh, saves me a git blame

AAndrii Sherman What is the purpose of mapping from JSON to an array of arrays at a code level a...

Max (@rozenmd)•5/15/23, 1:12 PM

good question, this should work then?

AAndrii Sherman I may be incorrect in any statement I make. However, based on my understanding,...

Max (@rozenmd)•5/15/23, 1:38 PM

I remember at some point you folks were using .all().all() instead of

.raw()

.raw()

do you happen to remember why it changed?

Max (@rozenmd)•5/15/23, 1:53 PM

considering .all doesn't do any remapping, and both .all and .raw call the same /query/query endpoint - maybe using .all.all is the fix?

Max (@rozenmd)•5/15/23, 1:53 PM

(we still need to figure out a fix for .raw on our side, but this might unblock y'all)

MMax (@rozenmd)considering .all doesn't do any remapping, and both .all and .raw call the same ...

divby0OP•5/16/23, 4:00 AM

@Andrii Sherman is that something we as users of drizzle can switch ourselves or would that require work on your end? I'd be eager to try it, currently I am still waiting to implement db stuff because it's a headache with joins

Ssourabh.sashank What is wrong in the below prepared statement for writing IN Query? var id = '...

Matt Silverlock•5/16/23, 10:47 AM

id

id

is a string literal and so you’re querying for WHERE CenterId IN (‘1,2,3')WHERE CenterId IN (‘1,2,3')

not an array.

Ssourabh.sashank var id = [1, 2, 3]; var stmt = globalEnv.DB.prepare('SELECT * FROM p...

Matt Silverlock•5/16/23, 11:40 AM

Effectively immeasurable: certainly for the array size here. D1's (SQLite's) JSON functions are very fast.

Original message was deleted

Matt Silverlock•5/16/23, 11:58 AM

WIP: https://silverlock-d1-indexes-json-l.cloudflare-docs-7ou.pages.dev/d1/learning/querying-json/#expanding-arrays-for-in-queries

Max (@rozenmd)•5/16/23, 2:39 PM

Almost all exception caught in D1 coreexception caught in D1 core errors are due to the nature of the D1 alpha unfortunately

Farhian•5/16/23, 6:06 PM

What would be the recommended column type for storing date time values? Thanks!

Farhian•5/16/23, 6:06 PM

(and format if relevant)

Original message was deleted

Farhian•5/16/23, 6:07 PM

Thank you!

Farhian•5/16/23, 6:07 PM

lol yeah

Writing join queries with D1 is near impossible with this issue Alex pointed out

Similar Threads

Similar Threads

Similar Threads