On Sun, 2007-07-01 at 00:31 +0100, jamie wrote:
> On Fri, 2007-06-29 at 10:56 +0800, Jerry Tan wrote:
> > This patch add one API to tracker client,
> > so it is easy to query tracker daemon's busy/idle status by call it.
> > 
> > for example, a desktop applet can show different icons when tracker 
> > daemon is busy or idle.
> > 
> > 
> 
> its not quite right
> 
> the queue length can be zero and tracker can still be indexing. Also we
> overflow large queues into a temp table to preserve RAM (you will see it
> called as pending files in the code) so it can be zero but still have a
> large number of pending files
> 
> you really need to check for the file process thread to be sleeping
> instead and set a suitable variable. There is a tracker->status variable
> already in the code in trackerd.c to track the various states but its
> not used anywhere so I dont know if its up to date
> 
> everything else looks ok so feel free to send an update
> 
> thanks
> 
> jamie
> 

I also think it would be better to have a GetStatus method instead of an
IsBusy so it can return a string for each of the possible status.

For a client it might also be handy to know, if its indexing, the uri of
the entity being indexed. 

We will also need a dbus signal StatusChanged to be emitted too but that
can wait if you are busy

jamie


_______________________________________________
tracker-list mailing list
tracker-list@gnome.org
http://mail.gnome.org/mailman/listinfo/tracker-list

Reply via email to