-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Creates ConcurrencyOptions
pydantic model.
#15291
Creates ConcurrencyOptions
pydantic model.
#15291
Conversation
CodSpeed Performance ReportMerging #15291 will not alter performanceComparing Summary
|
…mit enforcement prior to attempting to acquire a slot.
…onsistent with other prefect-nomenclature.
…e more consistent with other prefect-nomenclature." This reverts commit 0a4940f.
…dle attempting to acquire a slot from a deploy gcl that doesn't exist.
…' of https://github.com/PrefectHQ/prefect into jean/oss-66-create-a-concurrency-options-pydantic-model
…' of https://github.com/PrefectHQ/prefect into jean/oss-66-create-a-concurrency-options-pydantic-model
…' of https://github.com/PrefectHQ/prefect into jean/oss-66-create-a-concurrency-options-pydantic-model
…' of https://github.com/PrefectHQ/prefect into jean/oss-66-create-a-concurrency-options-pydantic-model
@@ -878,7 +878,18 @@ def job_variables(self): | |||
) | |||
|
|||
concurrency_limit: Mapped[Union[int, None]] = mapped_column( | |||
sa.Integer, default=None, nullable=True | |||
sa.Integer, | |||
server_default=None, |
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.
did this server default get added w/o a db migration?
or well - why did this get added?
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.
Oh. TIL alembic doesn't pick up server_default changes by default so I guess this doesn't matter but I think this line isn't doing anything as the DDL has already been generated and the updated DDL to alter the column w/ a db-server-side default isn't tracked via migrations.
Adds two pydantic models
ConcurrencyLimitConfig
,ConcurrencyOptions
, and updatesDeployment.concurrency_limit
to be able to useConcurrencyLimitConfig
. Also adds a concurrecy_options column to the deployments table.Related: #14934, #15340
Checklist
<link to issue>
"mint.json
.