Jamie McCracken wrote:
> On Mon, 2008-07-28 at 10:33 +0100, Martyn Russell wrote:
> yes best done in daemon

OK, that can easily be added.

>>> 3) stop words in tracker-parser.c - why did you remove this?
>> If you mean tracker->stop_words then it wasn't used as far as I can see.
>> If you mean tracker-parser.c, it is now in src/libtracker-common.
> 
> http://svn.gnome.org/viewvc/tracker/trunk/src/trackerd/tracker-parser.c?view=markup
> 
> this uses stop words - the one in libtracker-common does not

Pasting that link doesn't really help me. I already looked at that file
before replying to you. The tracker->stop_words is either always NULL or
never used to insert anything, therefore, to me, it looked completely
pointless. Is this the member you were talking about?

> pls make sure you load the appropriate stop word list for the stemming
> langauge - see trunk for this

I *think* that Ivan actually fixed this yesterday. Ivan, care to comment?

>>> 4) index merging - where did this go? Without this indexing wont scale
>>> up (its slow updating a large index) + you will suffer from
>>> fragmentation and wasted disk space due to hashtable resize relocations
>>> (these are never recovered) - merging solves both these issues
>> I will look into this.

Ivan care to comment here. I think you discussed this yesterday with
Carlos right?

>> Thanks for your quick review Jamie.
> 
> np - im on holiday next few days but will look to review again next
> weekend

Yesterday Ivan and Phillip got quite a bit done. The tracker-search-tool
seems to be working again and we now clean up index files on --reindex
which we didn't do before. This was stopping the whole t-s-t from
working for me.

We are continuing to work on your TODO list this week.

-- 
Regards,
Martyn
_______________________________________________
tracker-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/tracker-list

Reply via email to