Yvinayak07

We have supported varbinary to datetime cast in Babelfish. ``` 1> select CAST(0x0000A47F000F896A AS DateTime); 2> go datetime ----------------------- 2015-04-19 00:56:34.060 (1 rows affected) ``` from BBF version: ``` Babelfish...

We have also supported sys.+ operator (concat) for varbinary in babaelfish. ``` 1> declare @random varbinary(5) = 0x4142434445 2> declare @result varbinary(max) = 0x 3> select cast(@result + @random as...

Nice finding. I don't think this divergence is intended. we should go ahead and fix this bug where we need to throw error for some special string literals.

> As discussed offline, we also need to fix the overflow conditions for addition and subtraction. As per: https://github.com/MicrosoftDocs/sql-docs/blob/live/docs/t-sql/data-types/precision-scale-and-length-transact-sql.md > > > In addition and subtraction operations, we need max(p1...

> Please add comment why we have added changes in both upgrade scripts for clarity of reviewers Added comments in description.

> Tests for sysname and UDT looks good to me. But tests for `MAX` typmod does not look sufficient enough to test the handling we added for `MAX` typmod. One...