Skip to content

Conversation

@uzhastik
Copy link
Contributor

@uzhastik uzhastik commented Aug 7, 2024

append database name to table name in bulk upsert
cover bulk upsert in system test

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@uzhastik
Copy link
Contributor Author

uzhastik commented Aug 8, 2024

PR is ready for review, thank you in advance $)

@uzhastik
Copy link
Contributor Author

still ready :)

@potiuk potiuk merged commit 6e22364 into apache:main Aug 14, 2024
@uzhastik uzhastik deleted the ydb_system_tests_bulk_upsert branch August 14, 2024 07:07
Artuz37 pushed a commit to Artuz37/airflow that referenced this pull request Aug 19, 2024
…ert in system test (apache#41303)

* use bulk upsert in system test

* fix bulk_upsert test
romsharon98 pushed a commit to romsharon98/airflow that referenced this pull request Aug 20, 2024
…ert in system test (apache#41303)

* use bulk upsert in system test

* fix bulk_upsert test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants