Fokko commented on issue #6370: AIRFLOW-5701: Don't clear xcom explicitly before execution URL: https://github.com/apache/airflow/pull/6370#issuecomment-546676541 Just for the record. @bolkedebruin technically you're correct. But I was being pragmatic. We don't use the `id` column, so that's why I've dropped it. The migration will look like: 1. Drop the existing index and PK 1. Drop the id column 1. Create the PK on the fields on which the index was before Recreating such an index can take quite a lot of time, so just using the existing table as is, would be in practice exactly the same. Only the newly initiated installations would not have any `id` column, which isn't used anywhere 👍 However technically, I know, you should not touch existing migrations.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
