search

pwm-project / pwm

pwm
Other
911 stars 250 forks source link

Other SQL Syntax Issue (not general support request) #248

Open spitefultowel opened 6 years ago

spitefultowel commented 6 years ago

Per my previous ticket, there was a change to Other SQL that has caused support for MySQL/MariaDB to break in at least the latest snapshot.

Running version pwmVersion="1.8.0-SNAPSHOT" pwmBuild="28700112" pwmBuildType="" xmlVersion="4" createTime="2017-09-27T14:04:04Z" modifyTime="2017-09-28T14:25:37Z" works with no issues (I just installed on the same exact system that was giving issues) while pwmVersion="1.8.0-SNAPSHOT" pwmBuild="30195566" pwmBuildType="" xmlVersion="4" createTime="2017-11-22T15:26:57Z" modifyTime="2017-11-30T15:08:19Z" has an issue with the SQL table creation. I'm unable to personally locate the SQL to check/validate the syntax.

Again, this is not a general configuration issue but a legitimate issue with the SQL code to run on "Other" DB functionality.

kafej commented 6 years ago

So what build is working ?

spitefultowel commented 6 years ago

I personally know that build #28700112 is working. I'm unsure of anything after that as I've not gone through each build one by one. I've also heard that #29900166 is working as well.

vikingsource commented 6 years ago

The issue is a missing comma! Each create table statement is missing the comma after the word "KEY":

CREATE table CLUSTER_STATE ( id VARCHAR(128) NOT NULL PRIMARY KEY, value TEXT )

I'm running build 31855017 from SNAPSHOT-2018-02-01T20:37:06Z. A workaround is to run a simple sql file that creates all the tables before you start the configuration. I've included the one I used for MariaDB 10.2

db.txt