#5072 BLOC Update.: Activities dont load in 640

Zarro Boogs per Child bugtracker at laptop.org
Tue Nov 27 05:36:18 EST 2007


#5072: Activities dont load in 640
------------------------+---------------------------------------------------
  Reporter:  yani       |       Owner:  tomeu            
      Type:  defect     |      Status:  new              
  Priority:  blocker    |   Milestone:  Update.1         
 Component:  datastore  |     Version:                   
Resolution:             |    Keywords:  Update.1? review?
  Verified:  0          |  
------------------------+---------------------------------------------------

Comment(by marco):

 Replying to [comment:12 tomeu]:
 > Model.apply() looks to me like it syncs from Model to Xapian, isn't it?
 If we are sync'ing from Xapian to Model, then we don't need to call
 apply().

 It matters in this case because our code assumes all custom properties are
 "string" and Xapian has a different opinion. If secore exposes type
 information maybe the right solution is to actually respect it. In any
 case we need to check and resolve the inconsistency immediately, we can't
 let it run until it chokes at some random point.

 > >  * For Ship.2 I'm likely going to recommend to *not* apply the custom
 property patch. Source should not be a string in first place, we can keep
 defining it as "text" in model.py until it's worth to break model
 compatibility. Anyway we should think a bit more about it tomorrow morning
 and make a call in the Journal meeting.
 >
 > Perhaps that's the best thing we can do, but I'm a bit worried of
 setting source as "text", as this will be indexed, reducing performance in
 some unknown measure. 'string' is wrong but 'text' looks definitely worst
 to me.

 I doubt it will have any user visible effects for the source property. Do
 you disagree?

-- 
Ticket URL: <http://dev.laptop.org/ticket/5072#comment:13>
One Laptop Per Child <http://dev.laptop.org>
OLPC bug tracking system



More information about the Bugs mailing list