Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

[wip] Add size definitions to all model definitions #498

Draft
wants to merge 41 commits into
base: master
Choose a base branch
from
Draft

[wip] Add size definitions to all model definitions #498

wants to merge 41 commits into from

Conversation

wild-endeavor
Copy link
Contributor

TL;DR

Please replace this text with a description of what this PR accomplishes.

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

How did you fix the bug, make the feature etc. Link to any design docs etc

Tracking Issue

Remove the 'fixes' keyword if there will be multiple PRs to fix the linked issue

fixes https://github.com/flyteorg/flyte/issues/

Follow-up issue

NA
OR
https://github.com/flyteorg/flyte/issues/

wild-endeavor and others added 22 commits December 2, 2022 18:39
@wild-endeavor
move things to stdlib
58ebbdf 
Signed-off-by: Yee Hing Tong <[email protected]>
@wild-endeavor
check mysql first since postgres has a default
49a2ec2 
Signed-off-by: Yee Hing Tong <[email protected]>
@wild-endeavor
lowered the column sizes enough to get the node execution event table to
66d556a 
pass migration.

Signed-off-by: Yee Hing Tong <[email protected]>
@eapolinario
merge master
2140721 
Signed-off-by: Eduardo Apolinario <[email protected]>
@eapolinario
WIP - mysql support
a160b2e 
Signed-off-by: Eduardo Apolinario <[email protected]>
@wild-endeavor
merge master
8aea307 
Signed-off-by: Yee Hing Tong <[email protected]>
@wild-endeavor
add new migrations
1fe4dd2 
Signed-off-by: Yee Hing Tong <[email protected]>
@wild-endeavor
undo all the size deletions
a68cc1c 
Signed-off-by: Yee Hing Tong <[email protected]>
@eapolinario
Revert "undo all the size deletions"
e6dbda9 
This reverts commit a68cc1c.

Signed-off-by: Eduardo Apolinario <[email protected]>
@eapolinario
Merge remote-tracking branch 'origin' into mysql
d6639ab 
Signed-off-by: eduardo apolinario <[email protected]>
Signed-off-by: Eduardo Apolinario <[email protected]>
@eapolinario
Add Fixup migrations
b83482a 
Signed-off-by: eduardo apolinario <[email protected]>
@eapolinario
Reapply migration changes.
0c9d2b4 
Signed-off-by: eduardo apolinario <[email protected]>
@eapolinario
Remove old cruft
8808346 
Signed-off-by: eduardo apolinario <[email protected]>
@eapolinario
Remove gorm changes from migration_models.go
4a1f80a 
Signed-off-by: eduardo apolinario <[email protected]>
@eapolinario
One more pass in setting size constraints
7a43adb 
Signed-off-by: eduardo apolinario <[email protected]>
@eapolinario
Revert "One more pass in setting size constraints"
31f0bff 
This reverts commit 7a43adb.

Signed-off-by: Eduardo Apolinario <[email protected]>
@eapolinario
Revert "Remove gorm changes from migration_models.go"
b089e27 
This reverts commit 4a1f80a.

Signed-off-by: Eduardo Apolinario <[email protected]>
@eapolinario
Revert "Remove old cruft"
39ee118 
This reverts commit 8808346.

Signed-off-by: Eduardo Apolinario <[email protected]>
@eapolinario
Revert "Reapply migration changes."
c0c25a0 
This reverts commit 0c9d2b4.

Signed-off-by: Eduardo Apolinario <[email protected]>
@eapolinario
Remove size statements from models
72b2a85 
Signed-off-by: eduardo apolinario <[email protected]>
@eapolinario
Bring back commented code and minor size changes
59d7b36 
Signed-off-by: Eduardo Apolinario <[email protected]>
@eapolinario
Special-case mysql in the case of the task_executions table.
8901ea2 
Signed-off-by: eduardo apolinario <[email protected]>
@eapolinario eapolinario changed the title [wip] Move db logic to stdlib [wip] Add size definitions to all model definitions Mar 30, 2023
eapolinario and others added 12 commits March 29, 2023 18:43
@eapolinario
Add separate index to signals table
e31ccd8 
Signed-off-by: eduardo apolinario <[email protected]>
@eapolinario
Fix one off errors.
8effaf7 
Signed-off-by: eduardo apolinario <[email protected]>
@wild-endeavor
not null couple columns
6e34716 
Signed-off-by: Yee Hing Tong <[email protected]>
@wild-endeavor
partial treatment of stuff
bcc2b9d 
Signed-off-by: Yee Hing Tong <[email protected]>
@wild-endeavor
add manual indices for resources and task executions
5ccb295 
Signed-off-by: Yee Hing Tong <[email protected]>
@eapolinario
Set prefix length in primary_alt index on task_executions table
6cae700 
Signed-off-by: eduardo apolinario <[email protected]>
@eapolinario
Separate creation of primary_alt indexes into their own migrations.
e7361cc 
Signed-off-by: eduardo apolinario <[email protected]>
@wild-endeavor
use latest stdlib branch
ef4f5e7 
Signed-off-by: Yee Hing Tong <[email protected]>
@wild-endeavor
propeller to 1.1.83
29c702c 
Signed-off-by: Yee Hing Tong <[email protected]>
@wild-endeavor
==
e8a0349 
Signed-off-by: Yee Hing Tong <[email protected]>
@eapolinario
Add statements to change the type of primary keys in the case of post…
ce1773a 
…gres.

Signed-off-by: eduardo apolinario <[email protected]>
@eapolinario
Merge master
2cb96ab 
Signed-off-by: eduardo apolinario <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@katrogan katrogan Awaiting requested review from katrogan katrogan will be requested when the pull request is marked ready for review katrogan is a code owner

@EngHabu EngHabu Awaiting requested review from EngHabu EngHabu will be requested when the pull request is marked ready for review EngHabu is a code owner

@kumare3 kumare3 Awaiting requested review from kumare3 kumare3 will be requested when the pull request is marked ready for review kumare3 is a code owner

@pmahindrakar-oss pmahindrakar-oss Awaiting requested review from pmahindrakar-oss pmahindrakar-oss will be requested when the pull request is marked ready for review pmahindrakar-oss is a code owner

@hamersaw hamersaw Awaiting requested review from hamersaw hamersaw will be requested when the pull request is marked ready for review hamersaw is a code owner

@eapolinario eapolinario Awaiting requested review from eapolinario eapolinario will be requested when the pull request is marked ready for review eapolinario is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wild-endeavor @eapolinario