fetchPerson to be able to use in a transaction

PostHog / plugin-server

Service to process and save PostHog events, supporting JS/TS plugins at that

https://posthog.com

8 stars 5 forks source link

fetchPerson to be able to use in a transaction #610

Closed tiina303 closed 3 years ago

tiina303 commented 3 years ago

Changes

Need to use fetchPerson in a transaction to be sure the person wasn't merged in the middle of fetching and updating or any other action.

Not sure if we need to SELECT FOR UPDATE in this case too or not.

Checklist

[ ] Updated Settings section in README.md, if settings are affected
[ ] Jest tests

posthog-bot commented 3 years ago

Hey @tiina303! 👋 This pull request seems to contain no description. Please add useful context, rationale, and/or any other information that will help make sense of this change now and in the distant Mars-based future.

tiina303 commented 3 years ago

Will move to the new merged repo