Florent's patch was merged in trunk (7.0) at revision 4386 revision-id:
[email protected]
Thanks for the excellent contribution!
PS: Don't expect dramatic performance improvement, the syscalls were only
consuming a negligible amount of time because the filesystem cache will hold
the negative dentries in memory. Therefore each directory visit did not result
in an expensive filesystem call, but in most cases a direct hit in the FS cache.
PPS: There was nothing holding back the actual merge, except resource
contention in our R&D department. Sorry for the delay!
** Changed in: openobject-server
Status: In Progress => Fix Released
** Changed in: openobject-server
Milestone: None => 7.0
--
You received this bug notification because you are a member of OpenERP
Indian Team, which is subscribed to OpenERP Server.
https://bugs.launchpad.net/bugs/928376
Title:
file_open is not safe and performs too many useless syscalls
Status in OpenERP Server:
Fix Released
Bug description:
It happens with 5.0 and certainly with 6.1 too.
When you update the list of modules, the application calls:
tools.file_open(terp_file)
And file_open will browse all the parents of the "addons_path" for a zip file
...
I've added a statement to trace the call to "open()" :
open('/home/florent/erpdemo/parts/openerp-server/bin/addons/account.zip',
'rb')
open('/home/florent/erpdemo/parts/openerp-server/bin/addons.zip', 'rb')
open('/home/florent/erpdemo/parts/openerp-server/bin.zip', 'rb')
open('/home/florent/erpdemo/parts/openerp-server.zip', 'rb')
open('/home/florent/erpdemo/parts.zip', 'rb')
open('/home/florent/erpdemo.zip', 'rb')
open('/home/florent.zip', 'rb')
open('/home.zip', 'rb')
open('/.zip', 'rb')
This behaviour is seen on module installation or upgrade too.
It is probably a security issue which impacts performance as well.
To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-server/+bug/928376/+subscriptions
_______________________________________________
Mailing list: https://launchpad.net/~openerp-india
Post to : [email protected]
Unsubscribe : https://launchpad.net/~openerp-india
More help : https://help.launchpad.net/ListHelp