Comment 2 for bug 944670

Revision history for this message
Danny Tamez (zematynnad) wrote :

Some discussion on the FK thing:

[Friday, June 01, 2012] [01:28:46 PM] <zematynnad> hey Anthony - I'm trying to understand why the debtags_id is used in application instead of just a FK to debtags, any ideas?
[Friday, June 01, 2012] [01:29:06 PM] <achuni> performance
[Friday, June 01, 2012] [01:29:14 PM] <zematynnad> I see
[Friday, June 01, 2012] [01:29:19 PM] <achuni> so that it doesn't need to perform a separate query each time it wants the debtags
[Friday, June 01, 2012] [01:29:24 PM] <achuni> into a many-to-many table
[Friday, June 01, 2012] [01:29:39 PM] <zematynnad> should I follow the same pattern for the blacklist and whitelist fields then?
[Friday, June 01, 2012] [01:30:02 PM] <achuni> so, I'd say yes
[Friday, June 01, 2012] [01:30:13 PM] <achuni> but I'm not entirely sure it's the best idea
[Friday, June 01, 2012] [01:30:39 PM] <achuni> (I'd say yess because I wrote the debtags_id stuff)
[Friday, June 01, 2012] [01:31:02 PM] <achuni> if you don't feel strongly against the idea, then yes