Cloudflare Developers•3y ago

You can also read our discussion earlier: https://discord.com/channels/595317990191398933/9920605818

You can also read our discussion earlier: d1-database

MMatt Silverlock You can also read our discussion earlier: https://discord.com/channels/595317990...

amorfati•6/20/23, 10:32 PM

sweet, thank you

amorfati•6/20/23, 10:33 PM

when I created a new local db just now, it said binding is

db

db

. Is there a way to change that by just changing

binding

binding

in wrangler config?

Dahoom•6/20/23, 10:57 PM

use the --local option

Dahoom•6/20/23, 10:58 PM

so something like

wrangler d1 execute --local --comand="SELECT NOTHING"

wrangler d1 execute --local --comand="SELECT NOTHING"

DDahoom so something like ``wrangler d1 execute --local --comand="SELECT NOTHING"``

Dahoom•6/20/23, 10:58 PM

Dahoom•6/20/23, 11:04 PM

you can use wrangler

Dahoom•6/20/23, 11:04 PM

i tested it and it works in my pages directory

Dahoom•6/20/23, 11:05 PM

even when running wrangler pages dev it will read the wrangler.toml file and load the DB inside it

Dahoom•6/20/23, 11:13 PM

goodluck

amorfati•6/21/23, 1:53 AM

Error: statement too long

Error: statement too long

what is the statement size limit?

Aamorfati `Error: statement too long` what is the statement size limit?

Matt SilverlockOP•6/21/23, 1:54 AM

https://developers.cloudflare.com/d1/platform/limits/

Limits · Cloudflare D1 docs

Many of these limits will increase during D1’s public alpha. Join the #d1-open-alpha channel in the Cloudflare Developer Discord to keep up to date …

Aamorfati `Error: statement too long` what is the statement size limit?

amorfati•6/21/23, 1:54 AM

from docs "100,000 bytes (100KB)" is that accounted for experimental backend?

Aamorfati from docs "100,000 bytes (100KB)" is that accounted for experimental backend?

Matt SilverlockOP•6/21/23, 1:54 AM

That is the experimental backend, yes.

Pprogramjm Are there any examples of using D1 in an Astro project?

itsmatteomanf•6/21/23, 7:36 AM

I’m using it, what do you want to know?

XxPhi any ETA on when d1 will be in beta or production ready? just say 10y if it's fee...

itsmatteomanf•6/21/23, 7:37 AM

It’s pretty good already and it’s very stable, I’m sure the plan would be beta this year, but I have no specific knowledge. We need to wait for Matt.

betim•6/21/23, 9:49 AM

any way to create a DB with the regular backend alongside another with the experimental backend, locally? some of our queries don't work with the experimental backend (after importing data from our existing DBs that use the regular backend) and wanted to debug locally..

betim•6/21/23, 9:50 AM

this is using wrangler 3.1.0

Original message was deleted

betim•6/21/23, 9:57 AM

I was afraid that'd be the answer.. any guidance on figuring out why the experimental backend doesn't like our queries? any high level differences in SQLite support between regular and experimental backend that I should be aware of?

Original message was deleted

Max (@rozenmd)•6/21/23, 10:00 AM

cause

cause

should not be deprecated, it's still there

Max (@rozenmd)•6/21/23, 10:01 AM

Original message was deleted

betim•6/21/23, 10:01 AM

yup, got that

Error: too many terms in compound SELECT

Error: too many terms in compound SELECT

(coming from a

select {condition} union all select {another condition}

select {condition} union all select {another condition}

).. I suppose I'm wondering what SQLite differences would account for this seeing that the query works in the regular backend but errors out in the experimental?

Bbetim yup, got that `Error: too many terms in compound SELECT` (coming from a `select ...

Max (@rozenmd)•6/21/23, 10:03 AM

The experimental backend doesn't run in a WASM container, so there are stricter security limits in place https://developers.cloudflare.com/d1/platform/limits/

MMax (@rozenmd)The experimental backend doesn't run in a WASM container, so there are stricter ...

betim•6/21/23, 10:04 AM

yup, all good on this front.. everything is within the limits on the query and data side

betim•6/21/23, 10:05 AM

is there a version difference in the underlying sqlite runtime between regular and experimental backend? wondering if the query parsing/execution happens differently in the (I'm suspecting) newer version of sqlite that the experimental backend uses..

kian•6/21/23, 10:07 AM

The experimental backend has

SQLITE_LIMIT_COMPOUND_SELECT

SQLITE_LIMIT_COMPOUND_SELECT

set to

betim•6/21/23, 10:08 AM

does the backend support modifying that at either creation-time or runtime?

Max (@rozenmd)•6/21/23, 10:08 AM

that'll be why

betim•6/21/23, 10:08 AM

I think our query needs 5

Max (@rozenmd)•6/21/23, 10:09 AM

We can set it in workerd: https://github.com/cloudflare/workerd/blob/cfe19fdd962c2898dcb4a4bd0c4e1b2a60949f74/src/workerd/util/sqlite.c%2B%2B#L645, I think a github issue in workerd would help

MMax (@rozenmd)We can set it in workerd: https://github.com/cloudflare/workerd/blob/cfe19fdd962...

betim•6/21/23, 10:11 AM

k, thanks all.. this prevents us from moving to the experimental backend, which we've been really looking forward to for perf. will log a ticket to make this adjustable

Bbetim k, thanks all.. this prevents us from moving to the experimental backend, which ...

Matt SilverlockOP•6/21/23, 10:13 AM

Not adjustable: we’re likely to just increase it. A minimal sample query that fails now, but would work at (e.g.) 5 would be great.

Original message was deleted

Matt SilverlockOP•6/21/23, 10:14 AM

I don’t remove .cause.cause in that

Original message was deleted

Matt SilverlockOP•6/21/23, 10:15 AM

The PR doesn’t remove it, and it’s not merged, so…

MMax (@rozenmd)We can set it in workerd: https://github.com/cloudflare/workerd/blob/cfe19fdd962...

betim•6/21/23, 10:26 AM

FYI, ticket logged https://github.com/cloudflare/workerd/issues/795

GitHub

🐛 Experimental backend too restrictive on compount SELECT statement...

The experimental D1 backend configuration seems to have set limits around the compound SELECT statements that align with the SQLite high-security guidelines, but conflict with fairly simple use cas...

vassbence•6/21/23, 11:03 AM

Is there a limit on the number of inserts in a batch?

Bbetim FYI, ticket logged https://github.com/cloudflare/workerd/issues/795

Matt SilverlockOP•6/21/23, 11:32 AM

Commented.

Vvassbence Is there a limit on the number of inserts in a batch?

Matt SilverlockOP•6/21/23, 11:33 AM

Not specific to INSERT, outside of any normal limits. “Thousands” works fine.

vassbence•6/21/23, 11:34 AM

ok so definitely higher than queues max batch size, thanks

Pprogramjm How do you go about testing/running locally?

itsmatteomanf•6/21/23, 2:36 PM

It's kind of a pain... as it's a pain regardless testing with D1 on Pages, you need to pass it through the cli.

itsmatteomanf•6/21/23, 2:36 PM

I moved most of what I can to separate APIs, tbh.

vassbence•6/21/23, 3:02 PM

I'd like to get some help with the batch API.

I have a queue consumer which gets a list of messages to process and write them to db. If a write fails in .batch().batch() an error is thrown and everything is rolled back. This is not what I'd like to do, rather I'd like to ack/retry each item based on the success attribute

results.forEach(({ success }, index) => {
  if (success) {
    batch.messages[index].ack();
    return;
  }

  batch.messages[index].retry();
});

results.forEach(({ success }, index) => {
  if (success) {
    batch.messages[index].ack();
    return;
  }

  batch.messages[index].retry();
});

Is this possible or I'll need to use .run().run()s inside try catches?

vassbence•6/21/23, 3:04 PM

guess i could a condition in the sql to not do the insert if the foreign key fails but that would then output success: true and i'd get false positives

vassbence•6/21/23, 3:11 PM

or maybe checking for the foreign key before running the batch is good enough

You can also read our discussion earlier: https://discord.com/channels/595317990191398933/9920605818

Similar Threads