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

Support purchases-not-null-uid.sql migration crdb. #992

Merged
merged 2 commits into from
Mar 22, 2023

Conversation

sesposito
Copy link
Member

@sesposito sesposito commented Mar 7, 2023

Fix purchases-not-null-uid.sql migration for crdb.
Address fk inconsistent naming across pg and crdb.
Split the migration in two files so that the alter statements are executed in two separate transactions (crdb fails to migrate if multiple alter table statements are made within the same transaction).
See https://stackoverflow.com/questions/68803747/encapsulating-a-drop-and-add-constraint-in-a-transaction.

Fix purchases-not-null-uid.sql migration up for crdb.
The migration failed because pg and crdb name fk constraints differently.
Set the migration as notransaction because crdb fails to apply it if
multiple alter table statements are made on the same fk within the
same transaction.
@sesposito sesposito requested a review from zyro March 7, 2023 23:09
@sesposito sesposito merged commit 235bed3 into master Mar 22, 2023
@sesposito sesposito deleted the spe-fix-crdb-migration branch March 22, 2023 10:36
deflinhec added a commit to deflinhec/nakama that referenced this pull request Mar 30, 2023
* master: (27 commits)
  Satori API integration fixes (heroiclabs#1005)
  Attempt to exchange Google's authorization code when given one instead of as a JWT (heroiclabs#997)
  manually revert protobufany and updategroup in swagger.json (heroiclabs#1003)
  Make haystack cursors consistent with listing (heroiclabs#994)
  Support purchases-not-null-uid.sql migration on crdb. (heroiclabs#992)
  Add runtimes satori integration (heroiclabs#993)
  Disconnect banned users (heroiclabs#1001)
  Add runtime http insecure flag (heroiclabs#1002)
  Add Go runtime GroupUpdate userID param (heroiclabs#996)
  Better handling of matchmaker operations while the interval process is running.
  Ensure direct message channel message listing is correctly scoped to participants only.
  Update get purchase/subscription signature (heroiclabs#990)
  Fix reversal of open/closed group listing filter (heroiclabs#989)
  Always log out a deleted user (heroiclabs#987)
  Listing returned vars (heroiclabs#986)
  Add tournament record delete runtime functions. (heroiclabs#977)
  Improve google refund polling handling. (heroiclabs#982)
  Console user and authentication improvements. (heroiclabs#978)
  Console session handling improvements. (heroiclabs#979)
  Fix google purchase upsert query. (heroiclabs#973)
  ...

# Conflicts:
#	console/ui/dist/prod-nt/index.html
#	console/ui/dist/prod/index.html
deflinhec added a commit to deflinhec/nakama that referenced this pull request Mar 30, 2023
* master: (27 commits)
  Satori API integration fixes (heroiclabs#1005)
  Attempt to exchange Google's authorization code when given one instead of as a JWT (heroiclabs#997)
  manually revert protobufany and updategroup in swagger.json (heroiclabs#1003)
  Make haystack cursors consistent with listing (heroiclabs#994)
  Support purchases-not-null-uid.sql migration on crdb. (heroiclabs#992)
  Add runtimes satori integration (heroiclabs#993)
  Disconnect banned users (heroiclabs#1001)
  Add runtime http insecure flag (heroiclabs#1002)
  Add Go runtime GroupUpdate userID param (heroiclabs#996)
  Better handling of matchmaker operations while the interval process is running.
  Ensure direct message channel message listing is correctly scoped to participants only.
  Update get purchase/subscription signature (heroiclabs#990)
  Fix reversal of open/closed group listing filter (heroiclabs#989)
  Always log out a deleted user (heroiclabs#987)
  Listing returned vars (heroiclabs#986)
  Add tournament record delete runtime functions. (heroiclabs#977)
  Improve google refund polling handling. (heroiclabs#982)
  Console user and authentication improvements. (heroiclabs#978)
  Console session handling improvements. (heroiclabs#979)
  Fix google purchase upsert query. (heroiclabs#973)
  ...

# Conflicts:
#	console/ui/dist/prod-nt/index.html
#	console/ui/dist/prod/index.html
deflinhec added a commit to deflinhec/nakama that referenced this pull request Mar 30, 2023
* master: (27 commits)
  Satori API integration fixes (heroiclabs#1005)
  Attempt to exchange Google's authorization code when given one instead of as a JWT (heroiclabs#997)
  manually revert protobufany and updategroup in swagger.json (heroiclabs#1003)
  Make haystack cursors consistent with listing (heroiclabs#994)
  Support purchases-not-null-uid.sql migration on crdb. (heroiclabs#992)
  Add runtimes satori integration (heroiclabs#993)
  Disconnect banned users (heroiclabs#1001)
  Add runtime http insecure flag (heroiclabs#1002)
  Add Go runtime GroupUpdate userID param (heroiclabs#996)
  Better handling of matchmaker operations while the interval process is running.
  Ensure direct message channel message listing is correctly scoped to participants only.
  Update get purchase/subscription signature (heroiclabs#990)
  Fix reversal of open/closed group listing filter (heroiclabs#989)
  Always log out a deleted user (heroiclabs#987)
  Listing returned vars (heroiclabs#986)
  Add tournament record delete runtime functions. (heroiclabs#977)
  Improve google refund polling handling. (heroiclabs#982)
  Console user and authentication improvements. (heroiclabs#978)
  Console session handling improvements. (heroiclabs#979)
  Fix google purchase upsert query. (heroiclabs#973)
  ...
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.

2 participants