[MDEV-27955] main.func_json_notembedded test fails on out-of-memory on MariaDB 10.6.7 Created: 2022-02-26 Updated: 2023-02-05 Resolved: 2022-03-17 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Tests |
| Affects Version/s: | 10.6.7 |
| Fix Version/s: | 10.6.8, 10.7.4, 10.8.3 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Otto Kekäläinen | Assignee: | Daniel Black |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | not-10.5 | ||
| Issue Links: |
|
||||||||||||||||||||||||||||
| Description |
|
I notice that builds of MariaDB 10.6.7 in Debian was failing due to multiple reasons. One failure not reported before was main.func_json_notembedded on arch mipsel/s390x/powerpc:
Full logs at
|
| Comments |
| Comment by Daniel Black [ 2022-02-27 ] | |||||||||||||||||||||||||||||||
|
8 bytes repeated 62.5 million times seems to need more than 500M. So its not an overallocation, its a normal lack of resources. I had noticed it failing on bb on s390x, and its not marked as a big test, so mark it as such. Thanks for identifying the OOM cause. | |||||||||||||||||||||||||||||||
| Comment by Michael Widenius [ 2022-03-15 ] | |||||||||||||||||||||||||||||||
|
The idea with the test is to check if json_array_append(@arr, '$[0]', 1) can be interrupted. That said, I am fine with the suggested patch, but better to mark the test 'big' too, to ensure it is not run on machines with little resources. | |||||||||||||||||||||||||||||||
| Comment by Michael Widenius [ 2022-03-15 ] | |||||||||||||||||||||||||||||||
|
Ok to push | |||||||||||||||||||||||||||||||
| Comment by Otto Kekäläinen [ 2023-01-21 ] | |||||||||||||||||||||||||||||||
|
main.func_json_notembedded still failed in 10.6.11 and 10.11.1 in Debian, see e.g. https://buildd.debian.org/status/fetch.php?pkg=mariadb-10.6&arch=ppc64&ver=1%3A10.6.11-2&stamp=1672785257&raw=0 However the test does not run out of memory now but crashes:
| |||||||||||||||||||||||||||||||
| Comment by Daniel Black [ 2023-01-21 ] | |||||||||||||||||||||||||||||||
|
Abort is from MTR as its took too long. There was some guidance somewhere on improving the test. |