Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(client): enable ITX support for DataProxy #15694

Closed
wants to merge 13 commits into from
Closed

Conversation

danstarns
Copy link
Contributor

@danstarns danstarns commented Oct 6, 2022

Adds DataProxy support for interactive transactions.

mini-proxy pr: https://github.com/prisma/mini-proxy/pull/36/files
internal doc: https://www.notion.so/prismaio/iTX-between-DP-and-the-Client-91c133e5b60d43dbbfa409fde2b430b3

How to run the tests?

  1. Clone the branch and follow the instructions here to 'spin up' the mini-proxy.

  2. Run the interactive transaction tests in this branch with:

cd ./packages/client && pnpm run test:functional --data-proxy --no-mini-proxy-server --no-mini-proxy-default-ca interactive-transactions

@danstarns danstarns requested review from a team and millsp and removed request for a team October 6, 2022 16:02
@danstarns danstarns marked this pull request as draft October 6, 2022 16:03
@danstarns danstarns marked this pull request as ready for review October 7, 2022 10:50
@danstarns danstarns requested review from aqrln and removed request for millsp October 7, 2022 10:58
@aqrln aqrln marked this pull request as draft October 24, 2022 13:27
@aqrln aqrln self-assigned this Oct 24, 2022
@aqrln aqrln changed the title feat(client): ITX(Interactive Transactions) for DataProxy feat(client): enable ITX support for DataProxy Oct 24, 2022
Comment on lines +34 to +36
// TODO: temporary hack; we need to store url on the client itx proxy instead and pass it to the engine similar to id
// (probably some opaque `engineMeta: unknown` field and downcast it here to avoid leaking implementation details)
const txBaseUrls = new Map<string, string>()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

todo: what the comment says

@aqrln aqrln added this to the 4.6.0 milestone Oct 25, 2022
@aqrln
Copy link
Member

aqrln commented Oct 26, 2022

New PR for the integration branch: #16005

@aqrln aqrln closed this Oct 26, 2022
@aqrln aqrln deleted the client/itx-pdp branch October 26, 2022 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants