{"action":"created","issue":{"url":"https://api.github.com/repos/owncloud/core/issues/20053","labels_url":"https://api.github.com/repos/owncloud/core/issues/20053/labels{/name}","comments_url":"https://api.github.com/repos/owncloud/core/issues/20053/comments","events_url":"https://api.github.com/repos/owncloud/core/issues/20053/events","html_url":"https://github.com/owncloud/core/pull/20053","id":113365173,"number":20053,"title":"Don't lock if we're only reading cache metadata","user":{"login":"icewind1991","id":1283854,"avatar_url":"https://avatars.githubusercontent.com/u/1283854?v=3","gravatar_id":"","url":"https://api.github.com/users/icewind1991","html_url":"https://github.com/icewind1991","followers_url":"https://api.github.com/users/icewind1991/followers","following_url":"https://api.github.com/users/icewind1991/following{/other_user}","gists_url":"https://api.github.com/users/icewind1991/gists{/gist_id}","starred_url":"https://api.github.com/users/icewind1991/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/icewind1991/subscriptions","organizations_url":"https://api.github.com/users/icewind1991/orgs","repos_url":"https://api.github.com/users/icewind1991/repos","events_url":"https://api.github.com/users/icewind1991/events{/privacy}","received_events_url":"https://api.github.com/users/icewind1991/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/owncloud/core/labels/3%20-%20To%20Review","name":"3 - To Review","color":"ededed"},{"url":"https://api.github.com/repos/owncloud/core/labels/in%20progress","name":"in progress","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/owncloud/core/milestones/48","html_url":"https://github.com/owncloud/core/milestones/9.0-current","labels_url":"https://api.github.com/repos/owncloud/core/milestones/48/labels","id":1196845,"number":48,"title":"9.0-current","description":"","creator":{"login":"cmonteroluque","id":879976,"avatar_url":"https://avatars.githubusercontent.com/u/879976?v=3","gravatar_id":"","url":"https://api.github.com/users/cmonteroluque","html_url":"https://github.com/cmonteroluque","followers_url":"https://api.github.com/users/cmonteroluque/followers","following_url":"https://api.github.com/users/cmonteroluque/following{/other_user}","gists_url":"https://api.github.com/users/cmonteroluque/gists{/gist_id}","starred_url":"https://api.github.com/users/cmonteroluque/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmonteroluque/subscriptions","organizations_url":"https://api.github.com/users/cmonteroluque/orgs","repos_url":"https://api.github.com/users/cmonteroluque/repos","events_url":"https://api.github.com/users/cmonteroluque/events{/privacy}","received_events_url":"https://api.github.com/users/cmonteroluque/received_events","type":"User","site_admin":false},"open_issues":515,"closed_issues":155,"state":"open","created_at":"2015-07-07T14:21:23Z","updated_at":"2015-10-26T14:47:38Z","due_on":"2016-02-09T05:00:00Z","closed_at":null},"comments":4,"created_at":"2015-10-26T14:17:19Z","updated_at":"2015-10-26T15:01:05Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/owncloud/core/pulls/20053","html_url":"https://github.com/owncloud/core/pull/20053","diff_url":"https://github.com/owncloud/core/pull/20053.diff","patch_url":"https://github.com/owncloud/core/pull/20053.patch"},"body":"Since the db ensures we get a proper metadata snapshot we don't need to lock if we're only reading from the database.\r\n\r\nMade possible by splitting `checkUpdate` so we can still lock correctly if we need to re-scan\r\n\r\nMakes PROPFIND a non-locking operation ([comparison](https://blackfire.io/profiles/compare/3472c9b3-b00a-4d6d-950d-2a2808787414/graph))\r\n\r\ncc @DeepDiver1975 @PVince81 @LukasReschke "},"comment":{"url":"https://api.github.com/repos/owncloud/core/issues/comments/151165953","html_url":"https://github.com/owncloud/core/pull/20053#issuecomment-151165953","issue_url":"https://api.github.com/repos/owncloud/core/issues/20053","id":151165953,"user":{"login":"icewind1991","id":1283854,"avatar_url":"https://avatars.githubusercontent.com/u/1283854?v=3","gravatar_id":"","url":"https://api.github.com/users/icewind1991","html_url":"https://github.com/icewind1991","followers_url":"https://api.github.com/users/icewind1991/followers","following_url":"https://api.github.com/users/icewind1991/following{/other_user}","gists_url":"https://api.github.com/users/icewind1991/gists{/gist_id}","starred_url":"https://api.github.com/users/icewind1991/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/icewind1991/subscriptions","organizations_url":"https://api.github.com/users/icewind1991/orgs","repos_url":"https://api.github.com/users/icewind1991/repos","events_url":"https://api.github.com/users/icewind1991/events{/privacy}","received_events_url":"https://api.github.com/users/icewind1991/received_events","type":"User","site_admin":false},"created_at":"2015-10-26T15:01:05Z","updated_at":"2015-10-26T15:01:05Z","body":"The behavior in relation to `files:scan` is unchanged since that doesn't write lock the files anyway so the read-lock in the getFileInfo wouldn't have triggered a lock excpetion"}}