Last Comment Bug 526418 - gloda plugin message indexing failures should not be fatal to the message
: gloda plugin message indexing failures should not be fatal to the message
Status: NEW
:
Product: MailNews Core
Classification: Components
Component: Database (show other bugs)
: Trunk
: All All
: -- normal (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-11-03 23:27 PST by Andrew Sutherland [:asuth]
Modified: 2015-09-30 06:28 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments

Description Andrew Sutherland [:asuth] 2009-11-03 23:27:24 PST
As discussed on https://bugzilla.mozilla.org/show_bug.cgi?id=465618#c29, any failure of a gloda attribute provider will result in a message as being marked bad.  While fundattr.js and explattr.js failures must be treated as fatal for the message (resulting it being marked bad), other attribute providers should not receive the same treatment.
Comment 1 Wayne Mery (:wsmwk, NI for questions) 2014-01-05 12:42:49 PST
(In reply to Andrew Sutherland (:asuth) from comment #0)
> As discussed on https://bugzilla.mozilla.org/show_bug.cgi?id=465618#c29, any
> failure of a gloda attribute provider will result in a message as being
> marked bad.  While fundattr.js and explattr.js failures must be treated as
> fatal for the message (resulting it being marked bad), other attribute
> providers should not receive the same treatment.

As in, it permanently is marked bad and won't be indexed at all?
With current code, do we/did we encounter such messages in the wild?
Comment 2 Andrew Sutherland [:asuth] 2014-01-06 07:07:46 PST
Right, bad messages are invisible to gloda.  Right now I can't recall any buggy reasons we'd do this, but we did explicitly introduce an exception BadItemContentsError that we throw when 'From' contains 0 or more than 1 authors.

Certainly in development and early gloda we did run into various bad messages which caused the introduction of the bad message marking.  mxr on GLODA_BAD_MESSAGE_ID for related code and comments.

Note You need to log in before you can comment on or make changes to this bug.