Closed
Bug 1293375
Opened 9 years ago
Closed 7 years ago
2,500 "Suboptimal indexes for the SQL statement" emitted from nsDiskCacheDeviceSQL.cpp during linux64 debug testing
Categories
(Core :: Networking, defect, P5)
Core
Networking
Tracking
()
RESOLVED
FIXED
mozilla60
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: erahm, Assigned: erahm)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
2.02 KB,
patch
|
mayhemer
:
review+
|
Details | Diff | Splinter Review |
> 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
Assignee | ||
Comment 1•9 years ago
|
||
Assignee | ||
Comment 5•9 years ago
|
||
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
Flags: needinfo?(dcamp)
Summary: 900 instances of "Suboptimal indexes for the SQL statement 0x7fdc6dc3d550 (http://mzl.la/1FuID0j)." emitted from storage/mozStoragePrivateHelpers.cpp during linux64 debug testing → 2,500 "Suboptimal indexes for the SQL statement" emitted from nsDiskCacheDeviceSQL.cpp during linux64 debug testing
Updated•9 years ago
|
Priority: -- → P3
Updated•8 years ago
|
Component: Storage → Networking
Flags: needinfo?(dcamp)
Product: Toolkit → Core
Comment 6•8 years ago
|
||
ni? @michal since he might have some idea as well.
Flags: needinfo?(michal.novotny)
Whiteboard: [necko-triaged]
Updated•8 years ago
|
Flags: needinfo?(michal.novotny) → needinfo?(honzab.moz)
![]() |
||
Comment 7•8 years ago
|
||
And Michal had an idea asking me :D Yes, I'm probably the right person.
This is appcache which we no longer maintain. We don't update the appcache code unless it's a serious stability or security issue to fix.
Please annotate the statement to disable the warning. Thanks.
Flags: needinfo?(honzab.moz)
Priority: P3 → P5
Assignee | ||
Comment 8•7 years ago
|
||
This annotates two SQL statements that are responsible for a large amount of
warning spam due to "suboptimal indexes." Given we no longer wish to update
appcache code we can just disable the warning with a commment instead.
Attachment #8949915 -
Flags: review?(honzab.moz)
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → erahm
Status: NEW → ASSIGNED
![]() |
||
Comment 9•7 years ago
|
||
Comment on attachment 8949915 [details] [diff] [review]
Annotate sql index warnings in nsDiskCacheDeviceSQL
Review of attachment 8949915 [details] [diff] [review]:
-----------------------------------------------------------------
if this is the annotation to shut it up, then let's do it.
thanks!
Attachment #8949915 -
Flags: review?(honzab.moz) → review+
Comment 10•7 years ago
|
||
Pushed by erahm@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1354d17a426b
Annotate sql index warnings in nsDiskCacheDeviceSQL. r=mayhemer
Comment 11•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in
before you can comment on or make changes to this bug.
Description
•