[jdbc] Upgrade H2 to 2.2.224

[wborn]

Upgrades H2 database from 1.4.191 to 2.2.224.

Add the ;NON_KEYWORDS=VALUE configuration option at the end of your H2 JDBC URL because the add-on uses 'value' (which is also a keyword) as column name, e.g.: jdbc:h2:./testH2;NON_KEYWORDS=VALUE

Also fixes some queries used by the command extension because H2 always uses upper case table names.

For the change log, see:

https://h2database.com/html/changelog.html

For migration notes, see:

https://h2database.com/html/migration-to-v2.html

Fixes #13115

---

I also wrote a small migration script using Bash to migrate databases from H2 1.4 to H2 2.2:

#!/usr/bin/env bash

set -eo pipefail

if [ $# -eq 0 ]; then
    echo "This script migrates an H2 1.4 database to H2 2.2"
    echo
    echo "Usage: ./$(basename "$0") <h2-1.4-database.mv.db>"
    echo "Example: ./$(basename "$0") /var/lib/openhab/database.mv.db"
    exit 1
fi

if [ ! -e $1 ]; then
    echo "File '$1' does not exist"
    exit 1
fi

if [[ "$1" != *.mv.db ]]; then
    echo "File '$1' is not an H2 database because it has no '.mv.db' extension"
    exit 1
fi

DATABASE_PATH=$(realpath $1 | sed 's#.mv.db##')

echo "Downloading H2 JARs"
wget -q -O /tmp/h2-1.4.191.jar https://repo1.maven.org/maven2/com/h2database/h2/1.4.191/h2-1.4.191.jar
wget -q -O /tmp/h2-2.2.224.jar https://repo1.maven.org/maven2/com/h2database/h2/2.2.224/h2-2.2.224.jar

echo "Exporting H2 1.4 database"
java -cp /tmp/h2-1.4.191.jar org.h2.tools.Shell -url "jdbc:h2:$DATABASE_PATH" -sql "SCRIPT BLOCKSIZE 1000 TO '/tmp/h2-1.4-export.sql'" >/dev/null

echo "Renaming old database to: $DATABASE_PATH.mv.db.old"
mv $DATABASE_PATH.mv.db $DATABASE_PATH.mv.db.old

echo "Restoring H2 2.2 database from export"
java -cp /tmp/h2-2.2.224.jar org.h2.tools.RunScript -url "jdbc:h2:$DATABASE_PATH;NON_KEYWORDS=VALUE" -script "/tmp/h2-1.4-export.sql" -options FROM_1X >/dev/null

echo "Removing temporary files"
rm -f /tmp/h2-1.4.191.jar /tmp/h2-2.2.224.jar /tmp/h2-1.4-export.sql

echo "Done!"

[jlaur]

Thanks for taking the time to do this upgrade! I'm sure this is great news to @wollyd, @dependabot and others. 🙂

Do you think it would be reasonable to add a small note somewhere around here to mention the need to upgrade the database?
https://github.com/openhab/openhab-distro/blob/678720a9db8f9a7cfa6bb5f1f0e51666eaa9dcc6/distributions/openhab/src/main/resources/bin/update.lst#L119

[wborn]

Do you think it would be reasonable to add a small note somewhere around here to mention the need to upgrade the database?

Yes that is also part of the upgrade plan. 🙂

[openhab-bot]

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/openhab-4-1-release-discussion/152252/129