[MDEV-20142] encryption.innodb_encrypt_temporary_tables failed in buildbot with wrong result Created: 2019-07-23 Updated: 2023-12-10 Resolved: 2023-12-10 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Encryption, Storage Engine - InnoDB, Tests |
| Affects Version/s: | 10.2, 10.3, 10.4, 10.5 |
| Fix Version/s: | 10.4.33, 10.5.24, 10.6.17, 10.11.7, 11.0.5, 11.1.4, 11.2.3 |
| Type: | Bug | Priority: | Major |
| Reporter: | Elena Stepanova | Assignee: | Marko Mäkelä |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Description |
|
http://buildbot.askmonty.org/buildbot/builders/kvm-asan/builds/1730
|
| Comments |
| Comment by Alice Sherepa [ 2020-07-15 ] | ||||||||||||||||||
|
the same on 10.4 http://buildbot.askmonty.org/buildbot/builders/kvm-bintar-centos6-x86/builds/1380/steps/mtr/logs/stdio | ||||||||||||||||||
| Comment by Marko Mäkelä [ 2020-07-28 ] | ||||||||||||||||||
|
I think that the test is inaccurate. Maybe the page replacement is not always guaranteed, and we should insert more data to guarantee page eviction from the buffer pool? One thing that the test is definitely missing is a read of the table t2. It reads from t1 after hopefully evicting some of its pages by populating another table t2. The buffer pool size is at the minimum (5MiB). That would host at most 320 pages at the default innodb_page_size=16k. The test is inserting 2×8192 rows into 3 tables. The leaf-level record payload size is about 6+6+7+200+200 and 6+6+7+100+200+200 bytes (419 and 519 bytes). That is 4,251,648 bytes for the larger table. With the page header and trailer overhead, maybe it should exceed the 5,242,880 bytes? I was able to repeat this problem very easily when I was diagnosing | ||||||||||||||||||
| Comment by Marko Mäkelä [ 2020-07-28 ] | ||||||||||||||||||
|
I pushed a fixup to 10.2 that I hope will make the failure disappear. But, I was unable to reproduce this failure in my environment today. Therefore, I am not closing this bug yet. | ||||||||||||||||||
| Comment by Marko Mäkelä [ 2023-12-10 ] | ||||||||||||||||||
|
This test still occasionally fails. If I ignore all the crashes on POWER due to a Linux kernel bug (
I recently found out that information_schema.global_status.variable_value is a character string, not an integer. An explicit cast to integer is needed if we want integer comparison semantics instead of character string (9<10 but '9'>'10'). |