Skip to content

MDEV-40384: innodb_gis.geometry fails on replay#5412

Merged
bsrikanth-mariadb merged 1 commit into
bb-12.3-MDEV-39368-test-replay-preview-treefrom
13.1-MDEV-40384-innodb_gis.geometry_fails_on_replay
Jul 24, 2026
Merged

MDEV-40384: innodb_gis.geometry fails on replay#5412
bsrikanth-mariadb merged 1 commit into
bb-12.3-MDEV-39368-test-replay-preview-treefrom
13.1-MDEV-40384-innodb_gis.geometry_fails_on_replay

Conversation

@bsrikanth-mariadb

Copy link
Copy Markdown
Contributor

The test had innodb_strict_mode turned OFF, when running the test. But, in the replay, it was enabled, which caused the creation of tables with KEY_BLOCK_SIZE=16 fail.

Solution is to record the innodb_strict_mode variable in the context, so that it gets used during the replay.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds "innodb_strict_mode" to the list of optimizer-related system variables in "sql/opt_context_store_replay.cc". It also updates the InnoDB optimizer context replay test and its corresponding result files to record and verify the optimizer context with this new variable included. I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread sql/opt_context_store_replay.cc
@bsrikanth-mariadb
bsrikanth-mariadb force-pushed the 13.1-MDEV-40384-innodb_gis.geometry_fails_on_replay branch from 5070945 to 534540a Compare July 24, 2026 08:33
Comment thread sql/opt_context_store_replay.cc Outdated

@Olernov Olernov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after the cosmetic fix mentioned above

The test had innodb_strict_mode turned OFF, when running the test. But,
in the replay, it was enabled, which caused the creation of tables with
KEY_BLOCK_SIZE=16 fail.

Solution is to record the innodb_strict_mode variable in the context, so
that it gets used during the replay.
@bsrikanth-mariadb
bsrikanth-mariadb force-pushed the 13.1-MDEV-40384-innodb_gis.geometry_fails_on_replay branch from 534540a to 6c97ad6 Compare July 24, 2026 10:14
@bsrikanth-mariadb
bsrikanth-mariadb merged commit 6c97ad6 into bb-12.3-MDEV-39368-test-replay-preview-tree Jul 24, 2026
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants