Skip to content

Move database version profiles to individual client POMs#1642

Merged
tsegismont merged 1 commit intoeclipse-vertx:4.xfrom
tsegismont:backport-1640-4x
Mar 30, 2026
Merged

Move database version profiles to individual client POMs#1642
tsegismont merged 1 commit intoeclipse-vertx:4.xfrom
tsegismont:backport-1640-4x

Conversation

@tsegismont
Copy link
Copy Markdown
Member

Backport #1640

…tx#1640)

* Move database version profiles to individual client POMs

Relocate database-specific testing profiles from root pom.xml to individual client modules.
CI now uses Maven's -pl/-am flags for targeted builds instead of profile-based module selection.

This commit decentralizes the Maven profile configuration, moving it from a monolithic root POM to the individual database client modules.

Some portions of this content were created with the assistance of Claude Code.

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>

* Disable fail-fast in CI matrix

Otherwise, we don't get information about a particular database type

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>

* Skip MySQLTLSTest when testing MySQL 5.6

MySQL 5.6 only supports TLS 1.0/1.1 which are disabled in Java 11+

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>

* Make MSSQLRule work with all tested MSSQL versions

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>

* Skip PostgreSQL 9 NULL type inference tests

PostgreSQL 9 cannot infer types from NULL for BigDecimal, temporal types
(LocalTime, OffsetTime, Temporal), and their array variants. Override
these test methods to skip on PostgreSQL 9 using assumeTrue checks.

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>

* Removed empty profiles block

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>

---------

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
@tsegismont tsegismont merged commit 7ed77b8 into eclipse-vertx:4.x Mar 30, 2026
18 checks passed
@tsegismont tsegismont deleted the backport-1640-4x branch March 30, 2026 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant