Skip to content
This repository was archived by the owner on Jul 15, 2023. It is now read-only.

Conversation

@tom-englert
Copy link
Contributor

Deleting the data base file does no longer break CC

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 I prefer to see this list the actual exception types which are known to be thrown for the cases you describe.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fully agree, but here the exceptions may come from the DB providers and are neither documented nor consistent between DB providers - you can only get them by trial and error using all known providers, but then the code may break with any unknown provider.

@sharwell
Copy link
Contributor

sharwell commented Jul 3, 2015

I haven't tested this, but from the comments I felt I clearly understood both the problem and the way it's being solved (it makes my day when I see this happen). I'm not sure whether or not my requested change is feasible. Hopefully someone with more SQL experience can verify that the code is the best (or good enough) solution to the described problem but otherwise 👍.

tom-englert referenced this pull request in SergeyTeplyakov/CodeContracts Jul 6, 2015
SergeyTeplyakov added a commit that referenced this pull request Jul 8, 2015
@SergeyTeplyakov SergeyTeplyakov merged commit 01376c2 into microsoft:master Jul 8, 2015
@SergeyTeplyakov SergeyTeplyakov added this to the VS14 Support milestone Jul 8, 2015
@tom-englert tom-englert deleted the CacheHandling branch July 9, 2015 17:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants