Type:
Bug
Priority:
Major
Resolution:
Duplicate
Affects Version/s:
10.2.2 , 10.3.0 , 10.4.0 , 10.5.0 , 10.6.0
2022-01-21 Update
This problem was solved under terms of MDEV-25904
Old description
Field_string::cmp() seems to do unnecessary work trimming trailing spaces:
int Field_string::cmp( const uchar *a_ptr, const uchar *b_ptr) const
{
size_t a_len, b_len;
if (mbmaxlen() != 1)
{
size_t char_len= Field_string::char_length();
a_len= field_charset()->charpos(a_ptr, a_ptr + field_length, char_len);
b_len= field_charset()->charpos(b_ptr, b_ptr + field_length, char_len);
}
else
a_len= b_len= field_length;
/*
We have to remove end space to be able to compare multi-byte-characters
like in latin_de 'ae' and 0xe4
*/
return field_charset()->strnncollsp(a_ptr, a_len,
b_ptr, b_len);
}
In absolute majority cases, the difference between strings is found in the very beginning of the compared strings. So doing charpos() on the two arguments, before passing them to the actual comparison function, looks like an inefficient waste of CPU.
A better approach would be to implement a new comparison function with this tentative API:
int strnncollsp_nchars(CHARSET_INFO *cs,
const char *s1, size_t len1,
const char *s2, size_t len2,
size_t nchars);
Internally, the exact virtial implementations of strnncollsp_nchars() would do the same with what strnncollsp() do in the same collation, but with an extra limit on "nchars".
This new function should also help to fix a bug in the similar code in InnoDB: see MDEV-25440 for details.
{"report":{"fcp":1135.3000001907349,"ttfb":299.2000000476837,"pageVisibility":"visible","entityId":98929,"key":"jira.project.issue.view-issue","isInitial":true,"threshold":1000,"elementTimings":{},"userDeviceMemory":8,"userDeviceProcessors":64,"apdex":0.5,"journeyId":"c355df2c-9508-42ec-9a8e-ae50ddecf0d8","navigationType":0,"readyForUser":1227.1000001430511,"redirectCount":0,"resourceLoadedEnd":1248.9000000953674,"resourceLoadedStart":307.5,"resourceTiming":[{"duration":343.5,"initiatorType":"link","name":"https://jira.mariadb.org/s/2c21342762a6a02add1c328bed317ffd-CDN/lu2cib/820016/12ta74/0a8bac35585be7fc6c9cc5a0464cd4cf/_/download/contextbatch/css/_super/batch.css","startTime":307.5,"connectEnd":0,"connectStart":0,"domainLookupEnd":0,"domainLookupStart":0,"fetchStart":307.5,"redirectEnd":0,"redirectStart":0,"requestStart":0,"responseEnd":651,"responseStart":0,"secureConnectionStart":0},{"duration":343.40000009536743,"initiatorType":"link","name":"https://jira.mariadb.org/s/7ebd35e77e471bc30ff0eba799ebc151-CDN/lu2cib/820016/12ta74/494e4c556ecbb29f90a3d3b4f09cb99c/_/download/contextbatch/css/jira.browse.project,project.issue.navigator,jira.view.issue,jira.general,jira.global,atl.general,-_super/batch.css?agile_global_admin_condition=true&jag=true&jira.create.linked.issue=true&slack-enabled=true&whisper-enabled=true","startTime":307.90000009536743,"connectEnd":0,"connectStart":0,"domainLookupEnd":0,"domainLookupStart":0,"fetchStart":307.90000009536743,"redirectEnd":0,"redirectStart":0,"requestStart":0,"responseEnd":651.3000001907349,"responseStart":0,"secureConnectionStart":0},{"duration":351.5,"initiatorType":"script","name":"https://jira.mariadb.org/s/0917945aaa57108d00c5076fea35e069-CDN/lu2cib/820016/12ta74/0a8bac35585be7fc6c9cc5a0464cd4cf/_/download/contextbatch/js/_super/batch.js?locale=en","startTime":308.10000014305115,"connectEnd":308.10000014305115,"connectStart":308.10000014305115,"domainLookupEnd":308.10000014305115,"domainLookupStart":308.10000014305115,"fetchStart":308.10000014305115,"redirectEnd":0,"redirectStart":0,"requestStart":308.10000014305115,"responseEnd":659.6000001430511,"responseStart":659.6000001430511,"secureConnectionStart":308.10000014305115},{"duration":426.5,"initiatorType":"script","name":"https://jira.mariadb.org/s/2d8175ec2fa4c816e8023260bd8c1786-CDN/lu2cib/820016/12ta74/494e4c556ecbb29f90a3d3b4f09cb99c/_/download/contextbatch/js/jira.browse.project,project.issue.navigator,jira.view.issue,jira.general,jira.global,atl.general,-_super/batch.js?agile_global_admin_condition=true&jag=true&jira.create.linked.issue=true&locale=en&slack-enabled=true&whisper-enabled=true","startTime":308.7000000476837,"connectEnd":308.7000000476837,"connectStart":308.7000000476837,"domainLookupEnd":308.7000000476837,"domainLookupStart":308.7000000476837,"fetchStart":308.7000000476837,"redirectEnd":0,"redirectStart":0,"requestStart":308.7000000476837,"responseEnd":735.2000000476837,"responseStart":735.2000000476837,"secureConnectionStart":308.7000000476837},{"duration":429.40000009536743,"initiatorType":"script","name":"https://jira.mariadb.org/s/a9324d6758d385eb45c462685ad88f1d-CDN/lu2cib/820016/12ta74/c92c0caa9a024ae85b0ebdbed7fb4bd7/_/download/contextbatch/js/atl.global,-_super/batch.js?locale=en","startTime":308.90000009536743,"connectEnd":308.90000009536743,"connectStart":308.90000009536743,"domainLookupEnd":308.90000009536743,"domainLookupStart":308.90000009536743,"fetchStart":308.90000009536743,"redirectEnd":0,"redirectStart":0,"requestStart":308.90000009536743,"responseEnd":738.3000001907349,"responseStart":738.3000001907349,"secureConnectionStart":308.90000009536743},{"duration":429.7000000476837,"initiatorType":"script","name":"https://jira.mariadb.org/s/d41d8cd98f00b204e9800998ecf8427e-CDN/lu2cib/820016/12ta74/1.0/_/download/batch/jira.webresources:calendar-en/jira.webresources:calendar-en.js","startTime":309.10000014305115,"connectEnd":309.10000014305115,"connectStart":309.10000014305115,"domainLookupEnd":309.10000014305115,"domainLookupStart":309.10000014305115,"fetchStart":309.10000014305115,"redirectEnd":0,"redirectStart":0,"requestStart":309.10000014305115,"responseEnd":738.8000001907349,"responseStart":738.8000001907349,"secureConnectionStart":309.10000014305115},{"duration":429.89999985694885,"initiatorType":"script","name":"https://jira.mariadb.org/s/d41d8cd98f00b204e9800998ecf8427e-CDN/lu2cib/820016/12ta74/1.0/_/download/batch/jira.webresources:calendar-localisation-moment/jira.webresources:calendar-localisation-moment.js","startTime":309.30000019073486,"connectEnd":309.30000019073486,"connectStart":309.30000019073486,"domainLookupEnd":309.30000019073486,"domainLookupStart":309.30000019073486,"fetchStart":309.30000019073486,"redirectEnd":0,"redirectStart":0,"requestStart":309.30000019073486,"responseEnd":739.2000000476837,"responseStart":739.2000000476837,"secureConnectionStart":309.30000019073486},{"duration":492.7999999523163,"initiatorType":"link","name":"https://jira.mariadb.org/s/b04b06a02d1959df322d9cded3aeecc1-CDN/lu2cib/820016/12ta74/a2ff6aa845ffc9a1d22fe23d9ee791fc/_/download/contextbatch/css/jira.global.look-and-feel,-_super/batch.css","startTime":309.40000009536743,"connectEnd":0,"connectStart":0,"domainLookupEnd":0,"domainLookupStart":0,"fetchStart":309.40000009536743,"redirectEnd":0,"redirectStart":0,"requestStart":0,"responseEnd":802.2000000476837,"responseStart":0,"secureConnectionStart":0},{"duration":430.09999990463257,"initiatorType":"script","name":"https://jira.mariadb.org/rest/api/1.0/shortcuts/820016/47140b6e0a9bc2e4913da06536125810/shortcuts.js?context=issuenavigation&context=issueaction","startTime":309.60000014305115,"connectEnd":309.60000014305115,"connectStart":309.60000014305115,"domainLookupEnd":309.60000014305115,"domainLookupStart":309.60000014305115,"fetchStart":309.60000014305115,"redirectEnd":0,"redirectStart":0,"requestStart":309.60000014305115,"responseEnd":739.7000000476837,"responseStart":739.6000001430511,"secureConnectionStart":309.60000014305115},{"duration":492.5,"initiatorType":"link","name":"https://jira.mariadb.org/s/3ac36323ba5e4eb0af2aa7ac7211b4bb-CDN/lu2cib/820016/12ta74/d176f0986478cc64f24226b3d20c140d/_/download/contextbatch/css/com.atlassian.jira.projects.sidebar.init,-_super,-project.issue.navigator,-jira.view.issue/batch.css?jira.create.linked.issue=true","startTime":309.80000019073486,"connectEnd":0,"connectStart":0,"domainLookupEnd":0,"domainLookupStart":0,"fetchStart":309.80000019073486,"redirectEnd":0,"redirectStart":0,"requestStart":0,"responseEnd":802.3000001907349,"responseStart":0,"secureConnectionStart":0},{"duration":430.39999985694885,"initiatorType":"script","name":"https://jira.mariadb.org/s/5d5e8fe91fbc506585e83ea3b62ccc4b-CDN/lu2cib/820016/12ta74/d176f0986478cc64f24226b3d20c140d/_/download/contextbatch/js/com.atlassian.jira.projects.sidebar.init,-_super,-project.issue.navigator,-jira.view.issue/batch.js?jira.create.linked.issue=true&locale=en","startTime":309.80000019073486,"connectEnd":309.80000019073486,"connectStart":309.80000019073486,"domainLookupEnd":309.80000019073486,"domainLookupStart":309.80000019073486,"fetchStart":309.80000019073486,"redirectEnd":0,"redirectStart":0,"requestStart":309.80000019073486,"responseEnd":740.2000000476837,"responseStart":740.2000000476837,"secureConnectionStart":309.80000019073486},{"duration":602.1000001430511,"initiatorType":"script","name":"https://jira.mariadb.org/s/d41d8cd98f00b204e9800998ecf8427e-CDN/lu2cib/820016/12ta74/1.0/_/download/batch/jira.webresources:bigpipe-js/jira.webresources:bigpipe-js.js","startTime":317,"connectEnd":317,"connectStart":317,"domainLookupEnd":317,"domainLookupStart":317,"fetchStart":317,"redirectEnd":0,"redirectStart":0,"requestStart":317,"responseEnd":919.1000001430511,"responseStart":919.1000001430511,"secureConnectionStart":317},{"duration":915.5,"initiatorType":"script","name":"https://jira.mariadb.org/s/d41d8cd98f00b204e9800998ecf8427e-CDN/lu2cib/820016/12ta74/1.0/_/download/batch/jira.webresources:bigpipe-init/jira.webresources:bigpipe-init.js","startTime":317,"connectEnd":317,"connectStart":317,"domainLookupEnd":317,"domainLookupStart":317,"fetchStart":317,"redirectEnd":0,"redirectStart":0,"requestStart":317,"responseEnd":1232.5,"responseStart":1232.5,"secureConnectionStart":317},{"duration":139.89999985694885,"initiatorType":"xmlhttprequest","name":"https://jira.mariadb.org/rest/webResources/1.0/resources","startTime":814.8000001907349,"connectEnd":814.8000001907349,"connectStart":814.8000001907349,"domainLookupEnd":814.8000001907349,"domainLookupStart":814.8000001907349,"fetchStart":814.8000001907349,"redirectEnd":0,"redirectStart":0,"requestStart":814.8000001907349,"responseEnd":954.7000000476837,"responseStart":954.7000000476837,"secureConnectionStart":814.8000001907349},{"duration":159.59999990463257,"initiatorType":"script","name":"https://jira.mariadb.org/s/d41d8cd98f00b204e9800998ecf8427e-CDN/lu2cib/820016/12ta74/e65b778d185daf5aee24936755b43da6/_/download/contextbatch/js/browser-metrics-plugin.contrib,-_super,-project.issue.navigator,-jira.view.issue,-atl.general/batch.js?agile_global_admin_condition=true&jag=true&jira.create.linked.issue=true&slack-enabled=true&whisper-enabled=true","startTime":1089.3000001907349,"connectEnd":1089.3000001907349,"connectStart":1089.3000001907349,"domainLookupEnd":1089.3000001907349,"domainLookupStart":1089.3000001907349,"fetchStart":1089.3000001907349,"redirectEnd":0,"redirectStart":0,"requestStart":1089.3000001907349,"responseEnd":1248.9000000953674,"responseStart":1248.9000000953674,"secureConnectionStart":1089.3000001907349},{"duration":251.10000014305115,"initiatorType":"script","name":"https://www.google-analytics.com/analytics.js","startTime":1128.7000000476837,"connectEnd":0,"connectStart":0,"domainLookupEnd":0,"domainLookupStart":0,"fetchStart":1128.7000000476837,"redirectEnd":0,"redirectStart":0,"requestStart":0,"responseEnd":1379.8000001907349,"responseStart":0,"secureConnectionStart":0}],"fetchStart":0,"domainLookupStart":0,"domainLookupEnd":0,"connectStart":0,"connectEnd":0,"requestStart":109,"responseStart":299,"responseEnd":315,"domLoading":302,"domInteractive":1320,"domContentLoadedEventStart":1320,"domContentLoadedEventEnd":1368,"domComplete":1658,"loadEventStart":1658,"loadEventEnd":1658,"userAgent":"Mozilla/5.0 AppleWebKit/537.36 (KHTML, like Gecko; compatible; ClaudeBot/1.0; +claudebot@anthropic.com)","marks":[{"name":"bigPipe.sidebar-id.start","time":1307.6000001430511},{"name":"bigPipe.sidebar-id.end","time":1308.4000000953674},{"name":"bigPipe.activity-panel-pipe-id.start","time":1308.6000001430511},{"name":"bigPipe.activity-panel-pipe-id.end","time":1309.6000001430511},{"name":"activityTabFullyLoaded","time":1390.5}],"measures":[],"correlationId":"bdef0c3f95cb17","effectiveType":"4g","downlink":10,"rtt":0,"serverDuration":111,"dbReadsTimeInMs":19,"dbConnsTimeInMs":30,"applicationHash":"9d11dbea5f4be3d4cc21f03a88dd11d8c8687422","experiments":[]}}
This refactoring is needed for fixing the InnoDB index corruption bug
MDEV-25440.