Prisma

I know that many upsert queries don't exist. Any good many(multiple) upsert examples?
I wrote the code like this Do you have any other good way?
code # 3 - Is there a problem with using promise.all ?

<@U02BW8973BJ> :wave:
If you care about rolling back all upserts in case of an error, then approach #2 is the way to go.

In approach #3 you will get a faster output but you need to perform the retry of failed items yourself with an error handling mechanism.

If you don’t have a lot of users, I would suggest going for #2.

<@UESADRKMY>  thanks for answering!
If the example in #2 is the most useful, most for-loop await can be replaced with $transaction .

<@UESADRKMY>
I don't know much about $transaction, so I have one more question.
Is it correct to rollback the entire $transaction if any query fails?

Depends on your use case. If the upserts are related to each other then you should rollback, otherwise it’s fine to not use a transaction :slightly_smiling_face:

If each is unrelated, is it okay to use promise.all (#3) ?

Does prisma manage the connection pool well? Does using promise.all work without problems?

Depending on the number of queries, can too many be a problem with a wait timeout?

Lots of questions. I'm sorry. :sweat_smile:

`Promise.all` shouldn’t be a problem but as I said, you need to retry the failed items yourself using a retry mechanism.

&gt; Depending on the number of queries, can too many be a problem with a wait timeout?
How many queries in this case?

Sorry, I understand what you said earlier.
&gt; `Promise.all` shouldn’t be a problem but as I said, you need to retry the failed items yourself using a retry mechanism.

What if a single record fails in `Promise.all`? The records after that will not be processed so you need to have a mechanism of retrying those.

If I do Promise.allSettled , I think I can filter successful and failed queries with a filter.

To retry considering query failure, using $transaction seems to be the easiest way.:smile:

If multiple upserts occur frequently, I thought it would be better for performance to use promise.allSettled .:thinking_face: