Change type of defualts for include_deferred (#33271)

We should change the defaults to client defaults for MSSQL compatibility The change in #32709 was not liked by MSSQL Co-authored-by: Ephraim Anierobi <splendidzigy24@gmail.com> (cherry picked from commit d989e9d)
apache · Aug 9, 2023 · 2ff8d4d · 2ff8d4d
1 parent b099803
commit 2ff8d4d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/airflow/migrations/versions/0128_2_7_0_add_include_deferred_column_to_pool.py b/airflow/migrations/versions/0128_2_7_0_add_include_deferred_column_to_pool.py
@@ -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():

diff --git a/docs/apache-airflow/img/airflow_erd.sha256 b/docs/apache-airflow/img/airflow_erd.sha256
@@ -1 +1 @@
-3420f98a9dfa230d76be4da57bc4a3641cb929b33b9eafe7ffa41fb4fcc4d283
+52b0e45032fb1c60274374e0114cf199dfb759adc0a4fe32964136347457506a