Add support for transactions about prisma-client-py

[RobertCraigie]

@caelx Unfortunately not. I will be working on this soon though as it is by far the most requested feature so far.

[RobertCraigie]

@danigosa Thanks for the detailed comment!

Do you expect or consider to use TransactionManager out of a purely context manager?

Yes, I imagine that there may be cases where using a context manager is awkward but I can't think of any specific examples off the top of my head.

Why the TransactionManager.aexit() do not call rollback when error?

That appears to be a complete oversight on my part, thanks for pointing that out! While implementing this feature I was focused on getting the interactions with the engine to work instead of the actual feature behaviour. I'm grateful this was still a WIP as that is a pretty big oversight...

Do you see any problem on working around the context manager interface this way?

I can't see any problems occurring with usage outside of a context manager. The context manager simply provides a more convenient interface for handling exit / error state. What you have there is essentially the same as what the context manager will be doing (minus the missing rollback bug) and I can't foresee the actual interface, tx.start(), tx.commit() and tx.rollback() changing anytime soon.

[RobertCraigie]

@danigosa Done 👍

[danigosa]

Thanks for the feedback!

The main reason we work around the context manager is because it wasn't calling rollback, I think that if this issue is addressed it can be re-written as simply:

async def db_tx_middleware(request: Request, call_next: Any) -> Response:
    """
    UNTESTED
    Middleware that creates a transaction per request/response cycle
    Every transaction is either commited or rollback on success/exception
    It stores state on `request.state` context store
    WIP: Preview TransactionManager feature in prisma-client-py@wip/transactions
    """
    async with prisma.tx() as tx:
           request.state.db = tx
           return await call_next(request)

And delegate error handling upstream in the framework chain.

For now the version using raw TransactionManager works like a charm, no issues at all opening and commiting/rolling back transactions

[danielweil]

Thanks for the great work. Is this still a work on progress? I would love to use this in my project

[RobertCraigie]

Yes this is still a work in progress, I'll update the WIP to the latest version and then you can try it out!

[danielweil]

Thanks!

[RobertCraigie]

@danielweil Just updated my WIP, you can try it out in your project by installing from the branch like so:

pip install -U git+https://github.com/RobertCraigie/prisma-client-py@wip/transactions

However, there are absolutely no guarantees this will actually work.

[danielweil]

Thanks, will try it out.

[RobertCraigie]

Let me know if you encounter any issues!

[danielweil]

I tried installing it with the command, without uninstalling my latest prisma version. It resulted in the error below:

[RobertCraigie]

I've never encountered that error before but it looks like it has something to do with setuptools.

Try running:

pip install -U pip
pip install -U setuptools

And then try again

[danielweil]

Worked by adding --no-use-pep517 to the pip command.

[danielweil]

Is there any docs commenting on how to use transactions? Or it is the same syntax as TS?

[RobertCraigie]

No docs yet, basically the same syntax as batching, here's an example:

async with client.tx() as transaction:
    user = await transaction.user.create({'name': 'Robert'})

[RobertCraigie]

You can have a look through the tests for more examples:

https://github.com/RobertCraigie/prisma-client-py/blob/wip/transactions/tests/test_transactions.py

[danielweil]

Thanks! I am facing an issue here, can you thing about it?

I use output in the schema file and outputs to './client'. WIth version 0.2.2, my module is being added correctly and it works importing it like this:

"from prisma.client import Client"

When I use the transaction version, after generating the client, it gives me the error:

Exception: ModuleNotFoundError: No module named 'prisma.client'. Troubleshooting Guide: https://aka.ms/functions-modulenotfound

Do you think there is something about it?

[RobertCraigie]

Are you sure you need to use a custom output in the first place? By default Prisma Client Python will generate itself to the same place it was installed to.

If you're importing from a custom output of './client' then you're going to need to import it like this:

from client import Client

[danielweil]

It outputs to:

Generated Prisma Client Python to ./prisma/client in 961ms

Sorry, trying to figure out here how to do it

[RobertCraigie]

Why are you using custom output?

You probably want to either remove custom output or change it to ../client.

[danielweil]

Yep, worked out this way! Thanks

Tried using tx and it gave me the following error:

'Client' object has no attribute 'tx'

I am using it this way:

[RobertCraigie]

@danielweil Have you added the preview feature flag to your schema? e.g.

generator client {
  provider               = "prisma-client-py"
  recursive_type_depth   = -1
  previewFeatures        = ["interactiveTransactions"]
}

[danielweil]

Worked Great!

[danielweil]

Robert, would you mind making a new branch where this work in progress in merged with the newest release?

[RobertCraigie]

@danielweil I've updated the wip/transactions branch to include the changes from the latest release.

[danigosa]

@danielweil I've updated the wip/transactions branch to include the changes from the latest release.

Hi, we are using internally this branch (not in production) for our platform to test out transactions, could you integrate the commits in master like this one #402 to forward the code to current?

[danigosa]

For now we are using the interface TransactionManager without the context manager, we've seen so far that it works just fine, although we do not understand why the context manager version is not calling rollback on error.

This is how in our dev/staging environments is working with fastAPI with the standard DB middleware (open session/transaction on request, commit or rollback in response), calling explicitly tx_manager.rollback() to Prisma:

def get_db(request: Request) -> Prisma:
    """Global db fastApi dependency"""
    return request.state.db


async def db_tx_middleware(request: Request, call_next: Any) -> Response:
    """
    Middleware that creates a transaction per request/response cycle
    Every transaction is either commited or rollback on success/exception
    It stores state on `request.state` context store
    WIP: Preview TransactionManager feature in prisma-client-py@wip/transactions
    """
    response = Response("Internal server error", status_code=500)
    tx = prisma.tx()  # TransactionManager

    try:
        request.state.db = await tx.start()  # Client within the tx for the request
        response = await call_next(request)
    except Exception as e:
        log.exception(f"Exception while transaction, ROLLBACK: {e}")
        await tx.rollback()
    else:
        await tx.commit()

    return response

@router.get(
    "/organizations",
    response_model=List[OrganizationWithoutRelations],
)
async def list_organizations(
    take: int = 10, db: Prisma = Depends(get_db)
) -> List[Organization]:
    return await db.organization.find_many(take=take)

Background: we have a pgbouncer as data source for prisma in transaction mode, although we could live without transactions inferface batching writes, it made it hard to understand db operations like old fashion ORM, so this new interface allows us to operate as the good old request/response pattern typical in most ORM.

Some thoughts/questions:

• Do you expect or consider to use TransactionManager out of a purely context manager?
• Why the TransactionManager.__aexit__() do not call rollback when error?
• Do you see any problem on working around the context manager interface this way?

[caelx]

I'm also very interested in using this feature has the rollback bug been fixed on the branch?