search

microsoft / go-sqlcmd

The new sqlcmd, CLI for SQL Server and Azure SQL (winget install sqlcmd / sqlcmd create mssql / sqlcmd open ads)
https://learn.microsoft.com/sql/tools/sqlcmd/go-sqlcmd-utility
MIT License
334 stars 58 forks source link

`sqlcmd delete` fails for a context that has a container, and the container has been manually deleted already #282

Closed stuartpa closed 1 year ago

stuartpa commented 1 year ago

We could do a couple of things

  1. Allow the delete to succeed, and log a warning
  2. Fail the delete, and require a --force (but right now --force is bound to the scenario around not deleteing a context if user data is present in the contianer)
JyotikaGargg commented 1 year ago

In My point of view , we should go with #1 option (Allow the delete to succeed, and log a warning), As the container has already been deleted for that context ,so there is no point of failing a delete in this case. We should definitely be explicit in our message about the scenario