Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Method CDbCommandBuilder:: createMultipleInsertCommandWithIgnore added #4545

Closed
wants to merge 4 commits into from
Closed

Conversation

apphp
Copy link
Contributor

@apphp apphp commented Jan 29, 2024

Added method CDbCommandBuilder:: createMultipleInsertCommandWithIgnore and implemented for MySQL and Postgres

Copy link

what-the-diff bot commented Jan 29, 2024

PR Summary

  • Addition of the new method in CDbCommandBuilder.php
    A new function, createMultipleInsertCommandWithIgnore, has been introduced into the CDbCommandBuilder.php file. This function lays down the steps to create a Multiple INSERT command - a key database operation that can add several entries at once - while simultaneously ignoring any conflicts that arise.

  • Enhancement of CMysqlCommandBuilder.php
    Similarly, the createMultipleInsertCommandWithIgnore function has also been incorporated into the CMysqlCommandBuilder.php file. This version not only creates a Multiple INSERT command but adds some database-specific handling to avoid conflicts. This change is designed to improve how the software communicates with MySQL databases, a popular database management system.

  • Update to CPgsqlCommandBuilder.php
    Lastly, the same function has made its way into the CPgsqlCommandBuilder.php file, where it is again used to generate a Multiple INSERT command while handling conflicts. This database-specific improvement is aimed at better interfacing with PostgreSQL, another widely-used database system.

@apphp apphp closed this Jan 29, 2024
@apphp apphp reopened this Jan 29, 2024
@marcovtwout
Copy link
Member

Sorry, but Yii 1 Yii is in maintainace mode, only receiving security and compatibility fixes. This PR looks like optional functionality, which is not something we are taking into account right now.

@apphp
Copy link
Contributor Author

apphp commented Jan 30, 2024

Get it, thanks!

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.

2 participants