Skip to content
Snippets Groups Projects

More "semantic-versioning" compatible check for back-end schema

Merged Timm Fitschen requested to merge f-schema-version into dev
All threads resolved!
1 file
+ 15
6
Compare changes
  • Side-by-side
  • Inline
@@ -175,6 +175,15 @@ class DatabaseConnectionPool {
@@ -175,6 +175,15 @@ class DatabaseConnectionPool {
logger.debug("####################");
logger.debug("####################");
}
}
 
/**
 
* Check the version of the SQL server's database.
 
*
 
* <p>Current behaviour: Major versions must match and the database's minor version must be at
 
* least the expected minor version as defined in {@link
 
* ServerProperties.KEY_MYSQL_SCHEMA_VERSION}.
 
*
 
* <p>@todo Patch versions are not handled yet.
 
*/
private static void checkVersion(final Connection con)
private static void checkVersion(final Connection con)
throws SQLException, ConnectionException, CaosDBException {
throws SQLException, ConnectionException, CaosDBException {
try {
try {
@@ -186,9 +195,9 @@ class DatabaseConnectionPool {
@@ -186,9 +195,9 @@ class DatabaseConnectionPool {
final String expected_minor = expected_version[1];
final String expected_minor = expected_version[1];
con.setReadOnly(false);
con.setReadOnly(false);
final PreparedStatement prepareStatement = con.prepareStatement("SELECT CaosDBVersion()");
final PreparedStatement prepared = con.prepareStatement("SELECT CaosDBVersion()");
try {
try {
final ResultSet executeQuery = prepareStatement.executeQuery();
final ResultSet executeQuery = prepared.executeQuery();
if (executeQuery.next()) {
if (executeQuery.next()) {
final String actual_version = executeQuery.getString(1).toLowerCase();
final String actual_version = executeQuery.getString(1).toLowerCase();
@@ -200,13 +209,13 @@ class DatabaseConnectionPool {
@@ -200,13 +209,13 @@ class DatabaseConnectionPool {
if (Integer.parseInt(actual_major.replace("v", ""))
if (Integer.parseInt(actual_major.replace("v", ""))
< Integer.parseInt(expected_major.replace("v", ""))) {
< Integer.parseInt(expected_major.replace("v", ""))) {
logger.error(
logger.error(
"Version of the MySQL/MariaDB schema is incompatible.\n\tExpected: {}.{}\n\tActual: {}\nPlease upgrade the mysql backend.\n\n",
"Version of the MySQL/MariaDB schema is incompatible.\n\tExpected: {}.{}\n\tActual: {}\nPlease upgrade the MySQL backend.\n\n",
expected_major,
expected_major,
expected_minor,
expected_minor,
actual_version);
actual_version);
} else {
} else {
logger.error(
logger.error(
"Version of the MySQL/MariaDB schema is incompatible.\n\tExpected: {}.{}\n\tActual: {}\nPlease upgrade the server.\n\n",
"Version of the MySQL/MariaDB schema is incompatible.\n\tExpected: {}.{}\n\tActual: {}\nPlease upgrade the CaosDB server.\n\n",
expected_major,
expected_major,
expected_minor,
expected_minor,
actual_version);
actual_version);
@@ -216,7 +225,7 @@ class DatabaseConnectionPool {
@@ -216,7 +225,7 @@ class DatabaseConnectionPool {
if (!Objects.equal(actual_minor, expected_minor)) {
if (!Objects.equal(actual_minor, expected_minor)) {
if (Integer.parseInt(actual_minor) < Integer.parseInt(expected_minor)) {
if (Integer.parseInt(actual_minor) < Integer.parseInt(expected_minor)) {
logger.error(
logger.error(
"Version of the MySQL/MariaDB schema is incompatible.\n\tExpected: {}.{}\n\tActual: {}\nPlease upgrade the mysql backend.\n\n",
"Version of the MySQL/MariaDB schema is incompatible.\n\tExpected: {}.{}\n\tActual: {}\nPlease upgrade the MySQL backend.\n\n",
expected_major,
expected_major,
expected_minor,
expected_minor,
actual_version);
actual_version);
@@ -225,7 +234,7 @@ class DatabaseConnectionPool {
@@ -225,7 +234,7 @@ class DatabaseConnectionPool {
}
}
} else {
} else {
logger.error(
logger.error(
"Version of the MySQL schema could not be determined. This probably means, that the version is too old.\n\tExpected: {}.{}\n\tActual: unknown\nPlease upgrade the mysql backend.\n\n",
"Version of the MySQL schema could not be determined. This probably means that the version is too old.\n\tExpected: {}.{}\n\tActual: unknown\nPlease upgrade the MySQL backend.\n\n",
expected_major,
expected_major,
expected_minor);
expected_minor);
System.exit(1);
System.exit(1);
Loading