updateOnDuplicate and Postgres 12

Issue Description

What are you doing?

I have the following code, when I run code, it creates all rows as in array without “updating” updateOnDuplicate - is not working for me

const atd=(_data, _table = options.bd.table)=>{
	const sequelize = new Sequelize(options.bd.db, options.bd.login, options.bd.password, options.bd.options);
	let Table = sequelize.define(_table, {
		rating: Sequelize.INTEGER,
		// more lines
		reviewID: Sequelize.STRING
	}, {freezeTableName: true});
	sequelize.sync().then(()=>{
		Table.bulkCreate(_data, {
			updateOnDuplicate: ['reviewID']
		});
	});
};

Tried to use:

updateOnDuplicate: ['reviewID', 'id', 'createdAt', 'updatedAt', 'rating'] // all
updateOnDuplicate: ['reviewID', 'id'] // need and main keys
updateOnDuplicate: ['reviewID', 'rating'] // my keys

What do you expect to happen?

I need to update rows if keys match and if they don’t match - add. Formula - I need: if(inTable.reviewID === _data.reviewID){update_line}else{create_line} Check

What is actually happening?

Creating new items(_data.length) in table

Additional context

https://imgur.com/ZPUqgwR https://imgur.com/a/jVeb9gx

Environment

• Sequelize version: 5.21.1
• Node.js version: v12.13.0.
• Operating System: щinows 10 pro last
• Postgres: 12.0

Issue Template Checklist

How does this problem relate to dialects?

• I think this problem happens regardless of the dialect.
• I think this problem happens only for the following dialect(s): postgres
• I don’t know, I was using PUT-YOUR-DIALECT-HERE, with connector library version XXX and database version XXX

Would you be willing to resolve this issue by submitting a Pull Request?

• Yes, I have the time and I know how to start.
• Yes, I have the time but I don’t know how to start, I would need guidance.
• No, I don’t have the time, although I believe I could do it if I had the time…
• No, I don’t have the time and I wouldn’t even know how to start.

[papb]

You have posted a code snippet that shows the problem but is not entirely self-contained (i.e. I can’t just copy-paste it and run it). Please provide a SSCCE (also known as MCVE/reprex) to allow contributors to work on this on a timely manner. I suggest using the sequelize-sscce repository for this.

[cjancsar]

@Vadyanga you need to follow the issue template as @sushantdhiman has mentioned.