bugfix: fix could not rollback when insert the table with multiple key

[l81893521]

• I have registered the PR changes.

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

fixes #6076

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

[codecov]

Codecov Report

Merging #6077 (b4bf7e9) into 2.x (f42fb72) will increase coverage by 0.09%.
The diff coverage is 94.11%.

Additional details and impacted files

@@             Coverage Diff              @@
##                2.x    #6077      +/-   ##
============================================
+ Coverage     49.29%   49.38%   +0.09%     
- Complexity     4800     4805       +5     
============================================
  Files           913      913              
  Lines         31677    31678       +1     
  Branches       3825     3826       +1     
============================================
+ Hits          15614    15645      +31     
+ Misses        14514    14487      -27     
+ Partials       1549     1546       -3     

Files	Coverage Δ
.../rm/datasource/exec/BaseTransactionalExecutor.java	74.09% <94.11%> (+13.15%)	⬆️

... and 6 files with indirect coverage changes

[funky-eyes]

为什么要移动这段代码位置？而不是在它后面增加新方法？
Why move this piece of code instead of adding the new method after it?

[l81893521]

because reordering the type of method it looks more beautiful.

[funky-eyes]

because reordering the type of method it looks more beautiful.

那为什么不是将getColumnNamesInSQLList 挪到上面?
So why not move getColumnNamesInSQLList to the top?

[l81893521]

3 method of getColumnNamesWithTablePrefix and 3 method of getColumnNameInSQL. I just put it together

[funky-eyes]

Suggested change

	/**
	/**
	* Gets column name in sql.
	*
	* @param columnName the column name
	* @return the column name in sql
	*/
	protected String getColumnNameInSQL(String columnName) {
	String tableAlias = sqlRecognizer.getTableAlias();
	return tableAlias == null ? columnName : tableAlias + "." + columnName;
	}
	/**
	* Gets column names in sql.
	*
	* @param columnNames the column names
	* @return
	*/
	protected List<String> getColumnNamesInSQLList(List<String> columnNames) {
	List<String> columnNameWithTableAlias = new ArrayList<>();
	for (String columnName : columnNames) {
	columnNameWithTableAlias.add(this.getColumnNameInSQL(columnName));
	}
	return columnNameWithTableAlias;
	}

[funky-eyes]

Suggested change

	/**
	* Gets column name in sql.
	*
	* @param columnName the column name
	* @return the column name in sql
	*/
	protected String getColumnNameInSQL(String columnName) {
	String tableAlias = sqlRecognizer.getTableAlias();
	return tableAlias == null ? columnName : tableAlias + "." + columnName;
	}

[funky-eyes]

Suggested change

	/**
	* Gets column names in sql.
	*
	* @param columnNames the column names
	* @return
	*/
	protected List<String> getColumnNamesInSQLList(List<String> columnNames) {
	List<String> columnNameWithTableAlias = new ArrayList<>();
	for (String columnName : columnNames) {
	columnNameWithTableAlias.add(this.getColumnNameInSQL(columnName));
	}
	return columnNameWithTableAlias;
	}

[funky-eyes]

LGTM