Skip to content

Change latest connector test to smoke test in MariaDB#19765

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
ebyhr:ebi/mariadb-test
Nov 22, 2023
Merged

Change latest connector test to smoke test in MariaDB#19765
ebyhr merged 1 commit intotrinodb:masterfrom
ebyhr:ebi/mariadb-test

Conversation

@ebyhr
Copy link
Copy Markdown
Member

@ebyhr ebyhr commented Nov 16, 2023

Description

Update the docker image version to remove support for old versions https://endoflife.date/mariadb

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

@cla-bot cla-bot bot added the cla-signed label Nov 16, 2023
@ebyhr ebyhr requested review from hashhar and wendigo November 16, 2023 00:02
Copy link
Copy Markdown
Member

@hashhar hashhar left a comment

Choose a reason for hiding this comment

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

FWIW in other connectors like Mongo, SingleStore we use SmokeTest for the latest version and full ConnectorTest for the older versions.

You can also convert TestMySqlLegacyConnectorTest.java to a smoketest but will need to copy testNativeQueryWithClause, testAlterTableRenameColumnToLongName, testRenameColumnName and testRenameColumn.

Also, update the docker image versions.
@github-actions github-actions bot added the docs label Nov 17, 2023
@ebyhr ebyhr changed the title Add smoke test and remove redundant connector test in MariaDB Change latest connector test to smoke test in MariaDB Nov 17, 2023
@ebyhr ebyhr requested a review from hashhar November 17, 2023 10:03
@ebyhr ebyhr merged commit c2fb0c6 into trinodb:master Nov 22, 2023
@ebyhr ebyhr deleted the ebi/mariadb-test branch November 22, 2023 11:56
@github-actions github-actions bot added this to the 434 milestone Nov 22, 2023
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