rusackas commented on PR #41994:
URL: https://github.com/apache/superset/pull/41994#issuecomment-5448109228

   @Abdulrehman-PIAIC80387 CI's failing across the board 
(postgres/mysql/sqlite/playwright/cypress) because `5f2a8b9c4d1e`'s 
`down_revision` points at `8f3a1b2c4d5e`, but `d24e6b0a9c7f` already claimed 
that as its parent on `master`. Alembic sees two heads and every 
migration-dependent job bombs with "Multiple head revisions." Needs a rebase 
onto current `master` with the down_revision updated.
   
   Also, Michael's ask from last month for a SIP on the SET NULL vs CASCADE 
semantics is still open even after you trimmed the scope. Given this rewrites 
ON DELETE behavior on ~60 FK constraints, I'd want his sign-off before this 
merges, not just a scope reduction in a reply.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to