Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support MariaDB #4804

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

support MariaDB #4804

wants to merge 1 commit into from

Conversation

soshial
Copy link
Contributor

@soshial soshial commented Jan 29, 2022

Otherwise crashes with ERROR: Main method error - org.mariadb.jdbc.Driver - ClassNotFoundException (... < DataManager:113 < *:90 < Context:298.

@tananaev
Copy link
Member

I thought MySQL driver is compatible. Can't you use it?

@soshial
Copy link
Contributor Author

soshial commented Jan 29, 2022

It never worked for me. Had to add this MariaDB JAR manually.

@tananaev
Copy link
Member

It worked for me before. What's the error?

@soshial
Copy link
Contributor Author

soshial commented Jan 29, 2022

Look at the first message, please.

@tananaev
Copy link
Member

tananaev commented Jan 29, 2022

That's the error when you are trying to use MariaDB driver, but what's the error when you try MySQL driver?

@soshial
Copy link
Contributor Author

soshial commented Jan 29, 2022

Where do you set this up?

@tananaev
Copy link
Member

@soshial
Copy link
Contributor Author

soshial commented Jan 29, 2022

Why don't we just make it work as it is supposed to: MariaDB with its corresponding dedicated Java lib?
I am sure there might be some subtle problems we don't know about.
I think it's safer to to use MariaDB lib.

@tananaev
Copy link
Member

I don't think it's a good argument. If there are problems, please let me know what the problems are and I'll be happy to merge it. Originally MariaDB was supposed to be fully compatible with MySQL tools. If that's still the case, it makes no sense to include two identical drivers. If it's not the case anymore, I'm happy to reassess.

@soshial
Copy link
Contributor Author

soshial commented Jan 30, 2022

I checked it out and traccar works OK with MariaDB via MySQL connector/J.

But I will stick with mariadb-java-client anyway. It might very well be more performant than the MySQL counterpart.

@tananaev
Copy link
Member

OK, I'm going to close it for now, but if you find some evidence that it's better, I'm happy to merge it later.

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.

None yet

2 participants