[FIXED] SQLStore: Unable to run against MySQL 8.0.2+

[kozlovic]

This is due to the use of a now reserved word. Need to be escaped
with the "`" character. This character is used only for mysql
driver, not postgres (otherwise, it would break).

Resolves #536

[kozlovic]

@camsjams Would really appreciate if you could try this fix with your MySQL 8.0.2 server. I have checked that it still works with 5.7 and also change still works with Postgres server. Thanks!

[coveralls]

Coverage decreased (-0.02%) to 90.806% when pulling 3b80e99 on fix_536 into 339bd7a on master.

[camsjams]

Thanks for quick fix! I am having trouble installing the right Go version I think, tried 1.6.2 and 1.9.1 but caught up on just building the code - which version of Go are you running locally?

Once I can get a local build I will let you know.

[kozlovic]

Running go 1.10, but you could use go 1.9.4. If you can let me know what issue exactly you have, I may be able to help you.
You may also find this useful: https://nats.io/documentation/streaming/nats-streaming-quickstart/
To get the branch with the fix, once you have cloned the repo, you would do

git checkout fix_536

Then you can build from there.

[camsjams]

It was the path issue (namely where I did my git checkout relative to the $GOPATH).

Following https://nats.io/documentation/streaming/nats-streaming-quickstart/ and #248 helped me get the server running.

Now with go v1.10.0 and nats@nats-streaming-server#3b80e99a6698dc88f142b0ed75bb0cd27b4104de I can successfully use MySQL 8.0.2 - thanks!

[kozlovic]

@camsjams Thanks for the confirmation, I am merging the PR then!