-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add release_stage and release_date to ActorDefinition table #10051
Add release_stage and release_date to ActorDefinition table #10051
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments but looks good overall.
airbyte-config/models/src/main/resources/types/StandardDestinationDefinition.yaml
Show resolved
Hide resolved
airbyte-config/models/src/main/resources/types/StandardSourceDefinition.yaml
Show resolved
Hide resolved
: Enums.toEnum(standardSourceDefinition.getReleaseStage().value(), | ||
io.airbyte.db.instance.configs.jooq.enums.ReleaseStage.class).orElseThrow()) | ||
.set(ACTOR_DEFINITION.RELEASE_DATE, standardSourceDefinition.getReleaseDate() == null ? null | ||
: LocalDate.parse(standardSourceDefinition.getReleaseDate().toString())) // TODO make sure this works |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this verified?
: Enums.toEnum(standardSourceDefinition.getReleaseStage().value(), | ||
io.airbyte.db.instance.configs.jooq.enums.ReleaseStage.class).orElseThrow()) | ||
.set(ACTOR_DEFINITION.RELEASE_DATE, standardSourceDefinition.getReleaseDate() == null ? null | ||
: LocalDate.parse(standardSourceDefinition.getReleaseDate().toString())) // TODO make sure this works |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as below
eed18bb
to
597a7d6
Compare
- migration to add columns - update json schema to add new fields - update API schema to include new fields in definition reads - update tests
9c44768
to
a556599
Compare
What
Issue: #9711
Internal slack conversation with some more context: https://airbytehq-team.slack.com/archives/C02TL38U5L7/p1643247936054200
We want to mark connectors with different release stages (Alpha, Beta, Custom, Generally Available), and eventually want to mark new connectors with a 'New' badge.
This PR introduces
release_date
andrelease_stage
as optional columns/fields on our connector definition table/schemas.How
create*Definition
handler methods tocreateCustom*Definition
because we now assume that any definition created via API should be marked withcustom
release_stageSide note: I have another PR that adds a 'tombstone' column to actor_definition. Subodh informed me about upgrading the
CONFIGS_DATABASE_MINIMUM_FLYWAY_MIGRATION_VERSION
for cloud releases, so I will be sure to handle that for both of my PRs that introduce new columns. (I will bundle these two PRs as a single release and then release them to Cloud together).Recommended reading order
馃毃 User Impact 馃毃
No user impact yet, this just introduces the columns and fields that will later be populated with release stage/date information through our connector seeds.