Skip to content

Fixing Object is already attached to session problem for Database Isolation Tests#41317

Merged
jscheffl merged 1 commit intoapache:mainfrom
bugraoz93:bugfix/41067/providers-fix
Aug 7, 2024
Merged

Fixing Object is already attached to session problem for Database Isolation Tests#41317
jscheffl merged 1 commit intoapache:mainfrom
bugraoz93:bugfix/41067/providers-fix

Conversation

@bugraoz93
Copy link
Contributor

@bugraoz93 bugraoz93 commented Aug 7, 2024

Related: #41067
Fixes already attached to the session problem with passing session object from the fixture.
I have tested with the same parameters running in the Special tests/Database isolation test CI and all seems like passing.


^ 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.

@boring-cyborg boring-cyborg bot added area:providers provider:cncf-kubernetes Kubernetes (k8s) provider related issues provider:databricks labels Aug 7, 2024
@bugraoz93 bugraoz93 marked this pull request as ready for review August 7, 2024 19:20
@bugraoz93 bugraoz93 changed the title Fixing Object is already attached to session problem for Database Isolation Mode Fixing Object is already attached to session problem for Database Isolation Tests Aug 7, 2024
@bugraoz93
Copy link
Contributor Author

I was going to merge the PR but I couldn't. I resolved the thread since you said good to go and directly searched the merge button, habits :( @jscheffl Could you please merge the PR when you have time? Many thanks!

@jscheffl jscheffl merged commit 25342fa into apache:main Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments