[Dev] Factory method for EmailAddress

[Morgen Sagen]

Agreed.  What I was trying to say in response to the statement "looking 
up existing items could impact performance of reading mail" is that 
there shouldn't be any performance hit when reading mail.  But perhaps 
when Donn says "reading mail" he means "downloading mail via IMAP" and 
not "rendering mail messages in the UI" as I am assuming.  In which 
case I am mistaken and yes there could be a slight performance hit 
doing EmailAddress lookups when downloading messages; however you 
probably  gain back that extra time by committing fewer EmailAddress 
items to the repository.

Anyway, +1 to EmailAddress factory.

On Aug 27, 2004, at 3:39 PM, Brian Kirsch wrote:

> The check should take place anywhere an EmailAddress is created 
> including IMAP downloads, Contacts, SMTP sending, and IMAP Account 
> configuration (replyToAddress).
>
> Brian Kirsch - Email Framework Engineer
> Open Source Applications Foundation
> 543 Howard St. 5th Floor 
> San Francisco, CA 94105 
> (415) 946-3056 
>
> On Aug 27, 2004, at 3:03 PM, Morgen Sagen wrote:
>
>>
>> On Aug 27, 2004, at 2:41 PM, Donn Denman wrote:
>>
>>> Dev Folks,
>>>
>>> I'm planning to add a factory method for creation/reusing 
>>> EmailAddress Items, and there are some architecture issues for 
>>> consideration.
>>>
>>> Currently we're creating a lot of EmailAddress Items on the fly now 
>>> that Email and Sharing are starting to work.  I'm thinking the 
>>> factory method will allow us to reuse existing EmailAddress items to 
>>> prevent build-up of duplicates.
>>>
>>> Architecture Issues:
>>> ------------------------------
>>> * Looking up existing items could impact performance of reading 
>>> mail, but I'll use Ted's latest queries, and we can add a parameter 
>>> to disable the lookup if it's not fast enough.
>>
>> Won't this lookup-for-existing-addresses happen as each email message 
>> is downloaded by the imap layer (i.e., in the background)?  That way, 
>> when it comes time for the UI to display an email message, you'll 
>> already have references to the appropriate EmailAddress items?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 2377 bytes
Desc: not available
Url : http://lists.osafoundation.org/pipermail/dev/attachments/20040827/9c3a1426/smime.bin