Closed Bug 134899 Opened 22 years ago Closed 22 years ago

Dependency checking could be optimized away for some types of bug change

Categories

(Bugzilla :: Email Notifications, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 71794

People

(Reporter: sballard, Assigned: preed)

Details

I'm not terribly familiar with bugzilla as a product, but while using it on
mozilla.org I've noticed what appears to be a significant inefficiency in the
way it processes bugs.

Suppose bug A depends on bug B. Suppose now I add a comment and cc myself to bug
B (or just add a comment). Both of these are extremely frequent use cases in my
experience, and therefore worth optimizing for.

Now, bugzilla happily goes and processes bug B, sending email to everyone on B's
cc list etc. So far so good. But then it *also* goes and spends what appears to
be a significant amount of time looking at the CC list for bug A before deciding
not to send email to anyone at all. This is especially pronounced if the bug in
question has lots of dependencies - it can take almost a minute for bugzilla to
finish deciding to do nothing with all these bugs.

As far as I know, there are *no* circumstances under which anyone attached to
bug A might see a comment or CC change to bug B. So if bugzilla simply noticed
that those were the only two fields changed, it could skip the whole "checking
for dependency changes on..." step and probably reduce server load quite a bit.

There are probably other fields that will never trigger a notification on
dependent bugs, but even just optimizing out comment-and-cc-only changes would
probably make a huge difference to server load.

Of course, if I'm wrong and there *is* a way that somebody might get notified of
comment or cc changes in a dependent bug, then this should be WONTFIXed.

*** This bug has been marked as a duplicate of 71794 ***
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → DUPLICATE
*** Bug 140155 has been marked as a duplicate of this bug. ***
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.