[GitHub] rob05c commented on issue #2249: Add CRConfig DS Modified fields

2018-05-14 Thread GitBox
rob05c commented on issue #2249: Add CRConfig DS Modified fields URL: https://github.com/apache/incubator-trafficcontrol/pull/2249#issuecomment-387898955 I'll also note, I strongly dislike SQL triggers, for myriad reasons; but in this case, it's the best, safest way to guarantee the

[GitHub] rob05c commented on issue #2249: Add CRConfig DS Modified fields

2018-05-09 Thread GitBox
rob05c commented on issue #2249: Add CRConfig DS Modified fields URL: https://github.com/apache/incubator-trafficcontrol/pull/2249#issuecomment-387898955 I'll also note, I strongly dislike SQL triggers, for myriad reasons; but in this case, it's the best, safest way to guarantee the

[GitHub] rob05c commented on issue #2249: Add CRConfig DS Modified fields

2018-05-09 Thread GitBox
rob05c commented on issue #2249: Add CRConfig DS Modified fields URL: https://github.com/apache/incubator-trafficcontrol/pull/2249#issuecomment-387898955 I'll also note, I strongly dislike SQL triggers; but in this case, it's the best, safest way to guarantee the fields are updated

[GitHub] rob05c commented on issue #2249: Add CRConfig DS Modified fields

2018-05-09 Thread GitBox
rob05c commented on issue #2249: Add CRConfig DS Modified fields URL: https://github.com/apache/incubator-trafficcontrol/pull/2249#issuecomment-387898955 I'll also note, I strongly dislike SQL triggers, for myriad reasons; but in this case, it's the best, safest way to guarantee the

[GitHub] rob05c commented on issue #2249: Add CRConfig DS Modified fields

2018-05-09 Thread GitBox
rob05c commented on issue #2249: Add CRConfig DS Modified fields URL: https://github.com/apache/incubator-trafficcontrol/pull/2249#issuecomment-387898955 I'll also note, I strongly dislike SQL triggers, for myriad reasons; but in this case, it's the best, safest way to guarantee the

[GitHub] rob05c commented on issue #2249: Add CRConfig DS Modified fields

2018-05-09 Thread GitBox
rob05c commented on issue #2249: Add CRConfig DS Modified fields URL: https://github.com/apache/incubator-trafficcontrol/pull/2249#issuecomment-387898530 Tested the database migrations to succeed and roll back, tested the database triggers to properly update the modified fields when