Detects primary key constraint name change when migrating #39

[tonykaralis]

• Adds missing condition for determining primary key schema change
• Assigns primary key constraint name when creating existing delta

[tonykaralis]

@oskardudycz mine is slightly different to yours. Some sql code what was this for?

[oskardudycz]

@tonykaralis, I didn't notice that there is an index name already in the other select. Your version is better 👍 Thanks for busting that! 🙂

[tonykaralis]

@oskardudycz Thank you appreciate it, although I have 2 failing tests now on this branch :/

[tonykaralis]

Perhaps we could rerun the tests once more? I haven't been able to reproduce the failing tests locally.

[oskardudycz]

I've rerun them. Let's see 😉

[oskardudycz]

@tonykaralis, it looks like we have some flakiness, a different set of tests failed, but also the main branch seems to have issues. I'm probably going to pull the trigger tomorrow morning.

[tonykaralis]

Thank you, much appreciated!!

[oskardudycz]

Pulling that in, as test flakiness seems to be unrelated to the pull request.

[tonykaralis]

Closes #39