Codecov Report

Merging #9573 into master will increase coverage by <.01%.
The diff coverage is 100%.

Removed code from another commit ( sorry about that ). Not sure what to do with mssql test suite failing. It doesn't seem like I've changed at that much and the error seems general. Would appreciate any and all feedback and suggestions.

@sushantdhiman I've added integration tests for mysql and postgres. In order to check for comment field I had to change describeTable statements for both dialects. I can change the mssql statement as well but my tests for mssql integration tests are all failing in docker environment and I've already spent some time exploring why. Since I'm running Ubuntu on my machine I was experiencing difficulties installing it locally.

I will work on documentation. Added an example and a note that it's only available in MySQL, PostgreSQL and MSSQL

Since I'm running Ubuntu on my machine I was experiencing difficulties installing it locally.

If you are using docker, you can just

$ cd /to/sequelize/dev/folder
$ docker-compose up -d mssql

# I recommend .only to run only partial test suite as MSSQL image on Ubnutu fails for transactions
$ DIALECT=mssql npm run test-docker-integration

Still needs a bit more work. I'll try to finish it up later today.

@sushantdhiman hopefully this will do. Let me know if I need to make any corrections. Thank you for all the help and feedback!

In migration process, addColumn and comment is not working

Postgres 10,

        return queryInterface.addColumn(
          {
            schema: 'project',
            tableName: 'scheduleAssigns'
          },
          'timeLogger',
          {
            type: Sequelize.JSONB,
            comment: 'Time Log on JSON',
            defaultValue: {}
          }
        )

Generated Query:

ALTER TABLE "project"."scheduleAssigns" ADD COLUMN "timeLogger" JSONB DEFAULT '{}' COMMENT Time Log on JSON;

I've got ERROR: syntax error at or near "COMMENT"

Should Be:

ALTER TABLE "project"."scheduleAssigns" ADD COLUMN "timeLogger" JSONB DEFAULT '{}'; COMMENT ON "project"."scheduleAssigns"."timeLogger" IS 'Time Log on JSON';

@theRichu PR you are referencing only adds comments as a part of create table statement. It will not work when modifying table definition. Could be added as a separate issue @sushantdhiman

Now that @theRichu has opened a new PR #10076 we can discuss there