Skip to content

MDEV-39777: Fixed flaky plugins.unix_socket test#5137

Open
grooverdan wants to merge 1 commit into
MariaDB:12.3from
grooverdan:MDEV-39777
Open

MDEV-39777: Fixed flaky plugins.unix_socket test#5137
grooverdan wants to merge 1 commit into
MariaDB:12.3from
grooverdan:MDEV-39777

Conversation

@grooverdan
Copy link
Copy Markdown
Member

The plugins.unix_socket test showed the previous command process.

This occurs as after MDEV-38019 the early result to the client results in the server side thread lingering for its cleanup.

The subsequent exection of peer_crd_test can observer the connection still in progress.

Used --ping in the peer_cred_test so that the second execution of the test wouldn't observer the lingering connection from the first.

@grooverdan grooverdan requested a review from sanja-byelkin May 28, 2026 03:12
@grooverdan grooverdan added the MariaDB Foundation Pull requests created by MariaDB Foundation label May 28, 2026
Copy link
Copy Markdown

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

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 a --ping command to the mysql-test/suite/plugins/t/unix_socket.test test file to ensure the previous statement finishes cleanup before reading the processlist. There are no review comments, so I have no feedback to provide.

The plugins.unix_socket test showed the previous command process.

This occurs as after MDEV-38019 the early result to the client
results in the server side thread lingering for its cleanup.

The subsequent of DROP USER can observer the connection
still in progress and hence outputing a warning.

Used --ping in the peer_cred_test so that on completion, there
is no connection active for the user.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

MariaDB Foundation Pull requests created by MariaDB Foundation

Development

Successfully merging this pull request may close these issues.

2 participants