Add support for MS Varbinary(MAX) (#1714)

[TylerBrinks]

Hi, I'm the maintainer of https://github.com/TylerBrinks/SqlParser-cs, a C# port of this library. I'm finally getting around to submitting my first PR for this library based on an issue raised in the C# project.

This change updates the Varbinary type to take a BinaryLength similar to Varchar and Nvarchar where IntegerLength and Max is added to the enum. This replaces the u64 parameter in order to allow both varbinary(50) and varbinary(max) syntas for MS Sql and Generic implementations.

[alamb]

a C# port of this library.

Very cool!

[alamb]

Thanks @TylerBrinks -- this looks good to me. It seems there is a CI failure but the code looks good 👍

Once the CI is fixed I think we can merge it

[TylerBrinks]

I think I got it right in the updated most recent push. Awaiting approval if it passes the criteria.

[TylerBrinks]

@alamb Anything else I need to address? I'm learning the standard Rust way of doing things, hoping I got it right based on the PR checks in place.

[alamb]

@alamb Anything else I need to address? I'm learning the standard Rust way of doing things, hoping I got it right based on the PR checks in place.

Nope -- sorry for the delay and thanks for the pink @TylerBrinks -- I just lost track of this PR

I gave it another look and it looks great to me ❤️

🚀