Skip to content

Conversation

alexcan97
Copy link

This PR addresses # fix the error of muc_rooms and muc_light_rooms

Proposed changes include:

  • describe the functionality changes
  • describe new or updated tests
  • describe changes to the documentation

fix the error of muc_rooms and muc_light_rooms
@chrzaszcz
Copy link
Member

Could you describe in more detail, what bug this PR is supposed to fix, and how to reproduce this issue?

@alexcan97
Copy link
Author

fixed the syntax mistake in the mysql.sql

to run the old script mysql.sql to create table on MySQL, you will see the error.

@chrzaszcz
Copy link
Member

chrzaszcz commented Jan 4, 2023

fixed the syntax mistake in the mysql.sql

to run the old script mysql.sql to create table on MySQL, you will see the error.

We are testing MongooseIM in CircleCI with MySQL 8.0.30, and there is no error. What is your MySQL version?


CREATE TABLE muc_rooms(
id SERIAL,
id BIGINT UNSIGNED NOT NULL AUTO_INCREMENT,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://dev.mysql.com/doc/refman/8.4/en/numeric-type-syntax.html
Please have a look SERIAL have same meaning.

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.

3 participants