fix: make a copy of the columns instead of modifying it in place

This avoid unintentional modifications, especially in the case of a wrapped tool.

PiperOrigin-RevId: 810175539
This commit is contained in:
Google Team Member
2025-09-23 15:34:43 -07:00
committed by Copybara-Service
parent 38bbde6d56
commit aef1ee97a5
+1 -1
View File
@@ -139,7 +139,7 @@ def _generate_sql_for_knn(
distance_type, ann=False
)
embedding_parameter = f"@{_GOOGLESQL_PARAMETER_QUERY_EMBEDDING}"
columns += [f"""{distance_function}(
columns = columns + [f"""{distance_function}(
{embedding_column_to_search},
{embedding_parameter}) AS {_DISTANCE_ALIAS}
"""]