#3374 NORM Untriag: Bogus duplicate datastore items

[Zarro Boogs per Child]

#3374: Bogus duplicate datastore items
------------------------+---------------------------------------------------
  Reporter:  bert       |       Owner:  bcsaller          
      Type:  defect     |      Status:  new               
  Priority:  normal     |   Milestone:  Untriaged         
 Component:  datastore  |     Version:  Git as of bug date
Resolution:             |    Keywords:                    
  Verified:  0          |  
------------------------+---------------------------------------------------
Changes (by tomeu):

 * cc: dcbw (added)
  * priority:  high => normal


Comment:

 Have looked into it and we are indexing things in two places in the DS:

 backingstore.py:521 (update_async)

 and

 backingstore.py:507 (_update_completion)

 Dan, why are we updating the index in _update_completion? Shouldn't we
 update first the index and then copy the file into the private storage?

 Have pushed a fix so we don't create that extra entry, but we are still
 updating the index twice per every async update, which we should avoid per
 performance reasons.

-- 
Ticket URL: <https://dev.laptop.org/ticket/3374#comment:1>
One Laptop Per Child <https://dev.laptop.org>
OLPC bug tracking system