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

chore(rethinkdb): RetroReflection: Phase 2 #9834

Merged
merged 25 commits into from
Jun 25, 2024

Conversation

mattkrick
Copy link
Member

Description

Migrates RetroReflection to PG.
Note: this takes about 40 mins! Lets do this after hours if possible.

@github-actions github-actions bot added the size/l label Jun 6, 2024
@mattkrick mattkrick changed the base branch from master to retroReflection-phase1e June 24, 2024 17:06
@github-actions github-actions bot added size/m and removed size/l labels Jun 24, 2024
@rafaelromcar-parabol
Copy link
Contributor

rafaelromcar-parabol commented Jun 25, 2024

Note: this takes about 40 mins! Lets do this after hours if possible.

Let's then do it either

  • Any week day between 11pm and 2am the next day UK time - 3pm to 6pm Pacific Time. The time were there are less writes to RethinkDB (and I use that as less traffic into our app) is around 1am UK / 5pm Pacific.
  • Weekends after Fridays 11pm UK / 3pm Pacific and until Sunday are just perfect, almost no writes. For these big tables I'm not against doing the release on weekends. I think this one, TimeLineEvent, MeetingMember and maybe some more would benefit from doing this.

Up to you. I can make myself available just in case.

Also, it might take less in production. Let's see how long it takes on Staging and then let's decide.

Base automatically changed from retroReflection-phase1e to master June 25, 2024 19:27
@mattkrick mattkrick merged commit ab1e3bd into master Jun 25, 2024
6 checks passed
@mattkrick mattkrick deleted the chore/retroReflection-phase2 branch June 25, 2024 21:53
@github-actions github-actions bot mentioned this pull request Jun 25, 2024
24 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants