[Asterisk-code-review] queue_log: Add alembic script for generate db table for queue_log (asterisk[master])

[Rodrigo Ramirez Norambuena]

Rodrigo Ramirez Norambuena has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/4012 )

Change subject: queue_log: Add alembic script for generate db table for queue_log
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.asterisk.org/c/asterisk/+/4012/2/contrib/ast-db-manage/queue_log/versions/4105ee839f58_create_queue_log_table.py 
File contrib/ast-db-manage/queue_log/versions/4105ee839f58_create_queue_log_table.py:

https://gerrit.asterisk.org/c/asterisk/+/4012/2/contrib/ast-db-manage/queue_log/versions/4105ee839f58_create_queue_log_table.py@21 
PS2, Line 21:         'queue_log',
> I'm not really a database guy, but I believe it is usually considered a good practice to have at lea […]
Why not to use a multi column unique index between time + queuename?

If we follow what you say should add a `id` column to the cdr too.



-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/4012
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I35b928a6251f9da9a1742b2cd14c63a00c3d0f0c
Gerrit-Change-Number: 4012
Gerrit-PatchSet: 2
Gerrit-Owner: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Name of user not set #1000019
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-Comment-Date: Wed, 20 Nov 2019 15:39:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20191120/64a4c3b1/attachment.html>