1293375 - 2,500 "Suboptimal indexes for the SQL statement" emitted from nsDiskCacheDeviceSQL.cpp during linux64 debug testing

Status: NEW

Description

[Eric Rahm [:erahm] (please no mozreview requests)]

> 884 WARNING: Suboptimal indexes for the SQL statement 0x7fdc6dc3d550 (http://mzl.la/1FuID0j).: file storage/mozStoragePrivateHelpers.cpp, line 114

This warning [1] shows up in the following test suites:

>    884 - desktop-test-linux64/debug-web-platform-tests-5 5

It shows up in 605 tests. A few of the most prevalent:

>    121 -        /html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/parsing.html
>     25 -        /html/infrastructure/urls/resolving-urls/query-encoding/utf-16le.html
>     25 -        /html/infrastructure/urls/resolving-urls/query-encoding/windows-1252.html
>     25 -        /html/infrastructure/urls/resolving-urls/query-encoding/utf-8.html
>     25 -        /html/infrastructure/urls/resolving-urls/query-encoding/windows-1251.html
>      5 -        /html/semantics/embedded-content/the-img-element/sizes/parse-a-sizes-attribute.html
>      5 -        /html/semantics/scripting-1/the-template-element/definitions/template-contents.html
>      4 -        /html/semantics/forms/form-submission-0/url-encoded.html
>      4 -        /html/semantics/document-metadata/the-base-element/base_multiple.html
>      4 -        /html/semantics/scripting-1/the-template-element/template-element/template-descendant-frameset.html

[1] https://hg.mozilla.org/mozilla-central/annotate/e78975b53563/storage/mozStoragePrivateHelpers.cpp#l114

Comment 1

[Eric Rahm [:erahm] (please no mozreview requests)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=064ea3fd1dfa

Comment 5

[Eric Rahm [:erahm] (please no mozreview requests)]

The majority of these appear to be from two SQL statements in nsDiskCacheDeviceSQL.cpp [1]:

>  1258 WARNING: Suboptimal indexes for the SQL statement `SELECT ClientID, ItemType FROM moz_cache WHERE Key = ? ORDER BY LastFetched DESC, LastModified DESC;` [1 sort operation(s)] (http://mzl.la/1FuID0j).: file storage/mozStoragePrivateHelpers.cpp, line 114
>  1258 WARNING: Suboptimal indexes for the SQL statement `SELECT ns.ClientID, ns.ItemType FROM  moz_cache_namespaces AS ns JOIN moz_cache_groups AS groups  ON ns.ClientID = groups.ActiveClientID WHERE ns.NameSpace <= ?1 AND ?1 GLOB ns.NameSpace || '*' ORDER BY ns.NameSpace DESC, groups.ActivateTimeStamp DESC;` [1 sort operation(s)] (http://mzl.la/1FuID0j).: file storage/mozStoragePrivateHelpers.cpp, line 114

Looks like dcamp added these back in 2008 in bug 442813. Dave would you mind taking a look at this or redirecting to someone more appropriate? The general rule here is either it's possibly a perf hit and we should add an index or we can annotate the statement to avoid the warning.

[1] https://dxr.mozilla.org/mozilla-central/rev/e78975b53563d80c99ebfbdf8a9fbf6b829a8a48/netwerk/cache/nsDiskCacheDeviceSQL.cpp#1275-1283