Skip to content

Commit

Permalink
Change type of defualts for include_deferred
Browse files Browse the repository at this point in the history
We should change the defaults to client defaults for MSSQL compatibility
The change in apache#32709 was not liked by MSSQL
  • Loading branch information
ephraimbuddy authored and potiuk committed Aug 9, 2023
1 parent 2d2a1d6 commit c524915
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,7 @@
def upgrade():
"""Apply add include_deferred column to pool"""
with op.batch_alter_table("slot_pool") as batch_op:
batch_op.add_column(
sa.Column("include_deferred", sa.Boolean, nullable=False, server_default=sa.false())
)
batch_op.add_column(sa.Column("include_deferred", sa.Boolean, default=False))


def downgrade():
Expand Down
2 changes: 1 addition & 1 deletion docs/apache-airflow/img/airflow_erd.sha256
Original file line number Diff line number Diff line change
@@ -1 +1 @@
3420f98a9dfa230d76be4da57bc4a3641cb929b33b9eafe7ffa41fb4fcc4d283
52b0e45032fb1c60274374e0114cf199dfb759adc0a4fe32964136347457506a

0 comments on commit c524915

Please sign in to comment.