Closed Bug 66576 Opened 24 years ago Closed 23 years ago

use batching to speed up multiple message delete or move

Categories

(SeaMonkey :: MailNews: Message Display, defect, P2)

Product:
SeaMonkey
Component:
MailNews: Message Display
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED DUPLICATE of bug 80897
Milestone:
Future

People

(Reporter: sspitzer, Assigned: sspitzer)

References

Details

(Keywords: perf, Whiteboard: [nsbeta1+ 2/21])

Attachments

(2 files)

patch
(deleted), patch
Details | Diff | Splinter Review
patch, version 2.
(deleted), patch
Details | Diff | Splinter Review 
patch coming soon, for review and test.
Reminder to myself, report the performance metrics for this when I test next
against our pre-Pierre results.
Keywords: perf
QA Contact: esther → stephend
accepting, here comes the patch.
Status: NEW → ASSIGNED
Attached patch patch (deleted) — — Splinter Review 

    
    

    
  
in my local tree, I'm trying out a slight variation of that patch, one where I
call endBatch() after we select the next message.  I'll attach it.

either one, I think I've found a small bug with tree batching.  I'll log a new
bug on that, and try to find some time to discuss it with hyatt.
Priority: -- → P2
Target Milestone: --- → mozilla0.8

    
    

    
  
marking 0.8, p2.

    
  
Keywords: nsbeta1
Whiteboard: [nsbeta1+]

    
    

    
  
Perhaps it's about time this patch gets a r=/sr=?

    
    

    
  
This can't be checked in until 66583 is fixed, so there's not much point in
reviewing this.

    
    

    
  
moving to mozilla0.9
Target Milestone: mozilla0.8 → mozilla0.9

    
    

    
  
Seth does this bug go away with the thread pane rewrite?

    
    

    
  
marking nsbeta1- and moving to future milestone.  I think this might get taken 
care of with the rewrite. If not and there's something we can do here to speed 
us up then we should still consider doing it.
Keywords: nsbeta1 → nsbeta1-
Whiteboard: [nsbeta1+] → [nsbeta1+ 2/21]
Target Milestone: mozilla0.9 → Future

    
    

    
  
*** Bug 83656 has been marked as a duplicate of this bug. ***

    
    

    
  
updating summary.
Summary: use tree batching apis to speed up multiple message delete or move → use batching to speed up multiple message delete or move

    
    

    
  
note, I'm sure those patches are way out of date.  we'd still like to do this.

even with outliner, multiple deletes are slow.

    
    

    
  
*** Bug 115220 has been marked as a duplicate of this bug. ***

    
    

    
  
*** Bug 73758 has been marked as a duplicate of this bug. ***

    
    

    
  
Is this a duplicate of Bug 80897 ?

    
    

    
  
It looks like it.

*** This bug has been marked as a duplicate of 80897 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE

    
    

    
  
verified dup.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: