[Asterisk-code-review] alembic: Add support for MS-SQL (asterisk[13])

[Kevin Harwell]

Kevin Harwell has posted comments on this change. ( https://gerrit.asterisk.org/6459 )

Change subject: alembic: Add support for MS-SQL
......................................................................


Patch Set 1:

There was some concern about this going into the code base, and what that means moving forward from a maintenance perspective. After some discussion it was decided that modifying the Asterisk alembic scripts in order to allow MS-SQL Server support would be acceptable. However, with the following caveat:

MS-SQL Server support in Asterisk is not officially supported and should be treated as-is. Future changes/additions to any alembic scripts may break things for MS-SQL Server.

That being said, as changes are made to the alembic scripts effort should, and will be made to try and not break things, but it's just not guaranteed. Also, patches of course will be accepted from anyone with an interest in keeping the MS-SQL support current.


-- 
To view, visit https://gerrit.asterisk.org/6459
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: comment
Gerrit-Change-Id: I4a399ba3eed41a33ce8cb294968ad340221580ee
Gerrit-Change-Number: 6459
Gerrit-PatchSet: 1
Gerrit-Owner: Florian Floimair <f.floimair at commend.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Comment-Date: Fri, 08 Sep 2017 15:51:59 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20170908/004c8a51/attachment.html>