Cloudflare Developers•12mo ago

@Max (@rozenmd)

Kkamama @Max (@rozenmd)

Max (@rozenmd)•1/24/25, 11:22 AM

what numbers are you seeing since 2025-01-07?

kamamaOP•1/24/25, 11:24 AM

now around 500ms - 1000ms

Max (@rozenmd)•1/24/25, 11:24 AM

We got a fair bit faster since

kamamaOP•1/24/25, 11:24 AM

from worker1 -> worker2 -> DB

kamamaOP•1/24/25, 11:25 AM

let me deploy the performance check right

Kkamama from worker1 -> worker2 -> DB

Max (@rozenmd)•1/24/25, 11:25 AM

what do you see in the colo field at https://cloudflare.com/cdn-cgi/trace ?
and whats the output of wrangler d1 info dbnamewrangler d1 info dbname ?

kamamaOP•1/24/25, 11:26 AM

HKG

kamamaOP•1/24/25, 11:26 AM

running_in_region │ WNAM

Max (@rozenmd)•1/24/25, 11:27 AM

That's why:

kamamaOP•1/24/25, 11:28 AM

i did try apac before

kamamaOP•1/24/25, 11:28 AM

similar

戴

戴きますReina•1/24/25, 12:24 PM

anyone have project with hono with d1 prisma? how get the url for the db1 database? for prisma migrate

戴戴きますReina anyone have project with hono with d1 prisma? how get the url for the db1 databa...

Max (@rozenmd)•1/24/25, 12:27 PM

this might help
https://www.prisma.io/docs/orm/overview/databases/cloudflare-d1#migration-workflows

MMax (@rozenmd)this might help https://www.prisma.io/docs/orm/overview/databases/cloudflare-d1#...

戴

戴きますReina•1/24/25, 12:38 PM

that the problem, it doesnt, no matter how i tried to get the url for d1 we cant get it, need via wrangler

戴

戴きますReina•1/24/25, 12:38 PM

how to migrate from 001 migrations --> 002 migrations if a table already exist, this error confusing via wrangler. all example is writter from empty table not an existed one

Max (@rozenmd)•1/24/25, 12:41 PM

your migrations shouldn't be trying to create a table that already exists, but you can update your SQL to CREATE TABLE IF NOT EXISTSCREATE TABLE IF NOT EXISTS to avoid the error. it sounds like there's something wrong with your setup if you already have tables and are trying to run migrations to create them, though.

Isaac McFadyen•1/24/25, 12:42 PM

^ migrations are not meant to be indempotent, instead there's usually a table created within the DB that stores the migration name/ID and whether it's already been applied

Isaac McFadyen•1/24/25, 12:42 PM

That way you don't re-run the same migration twice.

戴

戴きますReina•1/24/25, 12:44 PM

the only thing i can think rn is via db file

Original message was deleted

Isaac McFadyen•1/24/25, 12:44 PM

It looks like you're trying to create a unique/primary/foriegn key that already exists?

Isaac McFadyen•1/24/25, 12:44 PM

What's your SQL statement?

IIsaac McFadyen What's your SQL statement?

戴

戴きますReina•1/24/25, 12:46 PM

sql statement is generated via prisma cli

戴戴きますReina anyone have project with hono with d1 prisma? how get the url for the db1 databa...

戴

戴きますReina•1/24/25, 12:46 PM

like this

戴戴きますReina sql statement is generated via prisma cli

Isaac McFadyen•1/24/25, 12:46 PM

Does the Prisma CLI connect to your D1 DB to verify that the table doesn't already exist? It shouldn't be giving you SQL to create the table if the table already exists.

IIsaac McFadyen Does the Prisma CLI connect to your D1 DB to verify that the table doesn't alrea...

戴

戴きますReina•1/24/25, 12:48 PM

we using prisma migrate diff like in the docs,
https://www.prisma.io/docs/orm/overview/databases/cloudflare-d1

Cloudflare D1 | Prisma Documentation

Guide to Cloudflare D1

IIsaac McFadyen Does the Prisma CLI connect to your D1 DB to verify that the table doesn't alrea...

戴

戴きますReina•1/24/25, 12:48 PM

yup its should not generate create table, even we change manual still not work

戴戴きますReina yup its should not generate create table, even we change manual still not work

Isaac McFadyen•1/24/25, 12:52 PM

It's very hard to troubleshoot unless you give the (manually changed) statement. The error you shared indicates that it's trying to do something it shouldn't be.

IIsaac McFadyen It's very hard to troubleshoot unless you give the (manually changed) statement....

戴

戴きますReina•1/24/25, 12:52 PM

for now i think i am gonna restart all the d1 since its still the beginning huhu

IIsaac McFadyen It's very hard to troubleshoot unless you give the (manually changed) statement....

戴

戴きますReina•1/24/25, 12:54 PM

for example 002 migrations already make table "CREATE TABLE "Single" but if we change manual (cause cli not working) is like this "CREATE TABLE IF NOT EXIST "Single""

戴戴きますReina for example 002 migrations already make table "CREATE TABLE "Single" but if we c...

Isaac McFadyen•1/24/25, 12:55 PM

Is there any other statement in there that might be trying to create a key (that's what the error is)? Maybe a CREATE INDEXCREATE INDEX or similar?

IIsaac McFadyen Is there any other statement in there that might be trying to create a key (that...

戴

戴きますReina•1/24/25, 1:02 PM

nothing

Isaac McFadyen•1/24/25, 1:03 PM

An error is still returned if the table cannot be created because of an existing index, even if the "IF NOT EXISTS" clause is specified.

戴戴きますReina we using prisma migrate diff like in the docs, https://www.prisma.io/docs/orm/ov...

戴

戴きますReina•1/24/25, 1:03 PM

i got lock in here, if you read this if we use --from-schema-datamodel --from-url we cant since binding is via wrangler not url like postgress, we also cant use

Isaac McFadyen•1/24/25, 1:03 PM

Ah, actually, looks to be a SQLite thing.

Isaac McFadyen•1/24/25, 1:03 PM

TLDR if there's an index in the table you can't even use CREATE IF NOT EXISTS

IIsaac McFadyen TLDR if there's an index in the table you can't even use CREATE IF NOT EXISTS

戴

戴きますReina•1/24/25, 1:03 PM

wait let me check

Isaac McFadyen•1/24/25, 1:03 PM

From https://www.sqlite.org/lang_createtable.html

戴

戴きますReina•1/24/25, 1:04 PM

we need to reset all data inside database ?

戴戴きますReina we need to reset all data inside database ?

Isaac McFadyen•1/24/25, 1:05 PM

The root issue is that you're trying to create a table when one already exists... Prisma shouldn't be generating/running queries to create a table if it's already present.

Isaac McFadyen•1/24/25, 1:05 PM

You don't need to delete the table, just don't run the migration if it's already in the DB

IIsaac McFadyen The root issue is that you're trying to create a table when one already exists.....

戴

戴きますReina•1/24/25, 1:08 PM

yeah bro thats the problem

when we migrate the db/wrangler cant do it since already existed table

戴

戴きますReina•1/24/25, 1:08 PM

i am gonna move away for now, maybe too much work, thanks bro

戴戴きますReina yeah bro thats the problem 😂 when we migrate the db/wrangler cant do it since a...

Isaac McFadyen•1/24/25, 1:10 PM

No, what I mean is that you should never be running a migration twice.

Isaac McFadyen•1/24/25, 1:11 PM

That kinda defeats the point of migrations which is to bring your DB to a certain schema version.

IIsaac McFadyen No, what I mean is that you should never be running a migration twice.

戴

戴きますReina•1/24/25, 1:11 PM

the migrations is not even run, cause it throw error in the table

戴戴きますReina the migrations is not even run, cause it throw error in the table

Isaac McFadyen•1/24/25, 1:11 PM

It's trying to run the migration though, because otherwise there'd be no error.

Original message was deleted

Isaac McFadyen•1/24/25, 2:39 PM

That's what I mean. D1 is trying to apply the migration called 0002_create_position_table.sql but it's already been applied so it fails.

Isaac McFadyen•1/24/25, 2:39 PM

Migrations shouldn't be applied twice, the fact that it is indicates the table tracking which migrations have been applied isn't working.

IIsaac McFadyen That's what I mean. D1 is trying to apply the migration called 0002_create_posit...

戴

戴きますReina•1/24/25, 3:03 PM

Hmmm maybe you kinda confused, the 002 is still not been applied cause i checked the number of table still on migration 001, moving 002 failed cause of double by of Prisma generate other table. Thus is due to prisma migrate diff cli, we can't do beside from --from-empty

戴

戴きますReina•1/24/25, 3:03 PM

But we can't forgot this bro, i just restart all db at it works fine

@Max (@rozenmd)

Similar Threads

Similar Threads

Similar Threads