Skip to content

Add ability to return number of affected rows to SpannerQueryDatabaseInstanceOperator#55127

Merged
gopidesupavan merged 1 commit intoapache:mainfrom
VladaZakharova:cloud-spanner-op-fix
Sep 4, 2025
Merged

Add ability to return number of affected rows to SpannerQueryDatabaseInstanceOperator#55127
gopidesupavan merged 1 commit intoapache:mainfrom
VladaZakharova:cloud-spanner-op-fix

Conversation

@VladaZakharova
Copy link
Copy Markdown
Contributor

The current set of Spanner Airflow operators support DDL and DML statements, but neither return values from e.g. INSERTS nor queries.
With this change now the user will be able to get number of affected rows by the query as a return value from the SpannerQueryDatabaseInstanceOperator.


^ 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 airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added area:providers provider:google Google (including GCP) related issues labels Sep 1, 2025
Copy link
Copy Markdown
Member

@gopidesupavan gopidesupavan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@VladaZakharova
Copy link
Copy Markdown
Contributor Author

@potiuk Hey, can you please check ?:)

@gopidesupavan gopidesupavan merged commit a08b399 into apache:main Sep 4, 2025
79 checks passed
@gopidesupavan
Copy link
Copy Markdown
Member

just merged LGTM, probably it can be included in the next release i see there is currently one in progress

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

Labels

area:providers provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants