From c78f13cd45acfac462e5f9bb8f0a1cd4edeed020 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Knut=20Olav=20L=C3=B8ite?= Date: Mon, 10 Nov 2025 09:14:48 +0100 Subject: [PATCH] chore: exclude last_update from the THEN RETURN clause Columns that are assigned a PENDING_COMMIT_TIMESTAMP() value cannot be included in THEN RETURN clauses. --- samples/model.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/samples/model.py b/samples/model.py index a4af86e4..a8c86047 100644 --- a/samples/model.py +++ b/samples/model.py @@ -199,12 +199,16 @@ class TicketSale(Base): String(36), ForeignKey("singers.id", spanner_not_enforced=True) ) # Create a commit timestamp column and set a client-side default of - # PENDING_COMMIT_TIMESTAMP() An event handler below is responsible for + # PENDING_COMMIT_TIMESTAMP(). An event handler below is responsible for # setting PENDING_COMMIT_TIMESTAMP() on updates. If using SQLAlchemy # core rather than the ORM, callers will need to supply their own # PENDING_COMMIT_TIMESTAMP() values in their inserts & updates. + # + # Columns that use PENDING_COMMIT_TIMESTAMP() cannot be included in a + # THEN RETURN clause. last_update_time: Mapped[datetime.datetime] = mapped_column( spanner_allow_commit_timestamp=True, + spanner_exclude_from_returning=True, default=text("PENDING_COMMIT_TIMESTAMP()"), )