bug-global
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Added support for file list in single file update


From: Shigio YAMAGUCHI
Subject: Re: Added support for file list in single file update
Date: Thu, 26 Jun 2014 09:43:16 +0900

Thank you for the data.

> > It means that files with no prefix need stat()?
>
> Yes

I guess that most everyday updating are change.
If so, it means that you cannot reduce stat() greatly.

First of all, if both source files and tag files are
in the remote file system, why don't you invoke gtags
in the remote host?

> Modified batch operation without prefix:
> [1140]$ time gtags --single-update no-prefix.files
>
> real    0m7.145s
> user    0m1.438s
> sys     0m1.229s
>
> Modified batch with prefix:
> [1141]$ time gtags --single-update prefix.files
>
> real    0m7.081s
> user    0m1.496s
> sys     0m1.129s <-- reduction in time by avoiding stat (not
> significant though due to file system caching)
>
> => There is a visible benefit in batch processing of files

I do not understand what you are testing.
Would you please show me the content of 'prefix.files' and
'no-prefix.files'?

> Ran under valgrind and find 'strtol()' via calls to 'atoi()' as one of
> the biggest contributors to performance overheads. I am looking at
> storing the integer in DB and fetching it instead of storing the
> integer as char and having to convert it back to get fid. That will be
> a separate patch. Wish this was under git... (I will try to import it
> into git)

This seems another issue.
Would you please post it separately.
Thank you in advance.

--
Shigio YAMAGUCHI <address@hidden>
PGP fingerprint: D1CB 0B89 B346 4AB6 5663  C4B6 3CA5 BBB3 57BE DDA3

reply via email to

[Prev in Thread] Current Thread [Next in Thread]