Skip to content

STAND-139: Upgrade MariaDB4j to 3.3.1 and delete ReusableDB in Standalone Platform#95

Merged
dkayiwa merged 1 commit intoopenmrs:masterfrom
Abhisy2506:STAND-139-upgrade-mariadb
Jan 27, 2026
Merged

STAND-139: Upgrade MariaDB4j to 3.3.1 and delete ReusableDB in Standalone Platform#95
dkayiwa merged 1 commit intoopenmrs:masterfrom
Abhisy2506:STAND-139-upgrade-mariadb

Conversation

@Abhisy2506
Copy link
Contributor

@Abhisy2506 Abhisy2506 commented Jan 20, 2026

Problem:

https://openmrs.atlassian.net/browse/STAND-139

Description

The purpose of this PR is to upgrade MariaDB4j to version 3.3.1. This version already has native support for Windows file locking and data reuse, allowing us to remove the custom ReusableDB wrapper and use standard MariaDB4j API.

### Solution:

1.Updated mariadb4jVersion to 3.3.1 in the parent pom.xml and cleaned up redundant mariadb4j-core entries in sub-modules (pom-step-01 to 05).
2. Updated MariaDbController.java to use the native DB.newEmbeddedDB(config) API.
3. Deleted the ReusableDB.java class as its functionality is now natively handled by the upgraded library.

Screenshot 2026-01-22 011030

### Testing:

  1. Build: Executed mvn clean install from the root directory; all modules built successfully.
  2. Tests: Ran mvn test -Dtest=MariaDbControllerTest to verify that the embedded database starts, stops, and handles permissions correctly.
  3. Environment: Verified database persistence and startup logic on Windows.

@Muta-Jonathan
Copy link
Member

@Abhisy2506 could you update the Description to include the task u worked on ... you can take a look at this PR -> #91

@Abhisy2506 Abhisy2506 force-pushed the STAND-139-upgrade-mariadb branch from 22d4c66 to b3549aa Compare January 21, 2026 19:59
Copy link
Member

@Muta-Jonathan Muta-Jonathan left a comment

Choose a reason for hiding this comment

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

LGTM thanks @Abhisy2506
cc @dkayiwa

@dkayiwa
Copy link
Member

dkayiwa commented Jan 27, 2026

@Muta-Jonathan did you test and confirm that everything still works as expected with these changes?

@Muta-Jonathan
Copy link
Member

Muta-Jonathan commented Jan 27, 2026

yes @dkayiwa
Screenshot 2026-01-27 143125

The error in the loading is associated with the platform version not the changes in this PR just an add the changes here also work for linux and mac devices

@dkayiwa dkayiwa merged commit 01274d1 into openmrs:master Jan 27, 2026
3 checks passed
@dkayiwa
Copy link
Member

dkayiwa commented Jan 27, 2026

Thanks @Abhisy2506 and @Muta-Jonathan

@Abhisy2506
Copy link
Contributor Author

"Welcome @dkayiwa and @Muta-Jonathan. Thanks for the guidance and for helping me get my first PR successfully merged!"

@Muta-Jonathan Muta-Jonathan changed the title STAND-139: Upgrade MariaDB4j to 3.3.1 and delete ReusableDB STAND-139: Upgrade MariaDB4j to 3.3.1 and delete ReusableDB in Platform Jan 28, 2026
@Muta-Jonathan Muta-Jonathan changed the title STAND-139: Upgrade MariaDB4j to 3.3.1 and delete ReusableDB in Platform STAND-139: Upgrade MariaDB4j to 3.3.1 and delete ReusableDB in Standalone Platform Jan 28, 2026
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.

3 participants