henry3260 commented on code in PR #58344:
URL: https://github.com/apache/airflow/pull/58344#discussion_r2553949986
##########
airflow-core/src/airflow/api_fastapi/execution_api/routes/xcoms.py:
##########
@@ -88,111 +88,15 @@ async def xcom_query(
return query
[email protected](
Review Comment:
I modified the key:path in the router definition to address unintended
greedy matching issues in some routes. Previously, these issues caused
conflicts that affected routes associated with XComs, as broader path
definitions were overriding more specific ones.
To resolve this, I reordered the router definitions to prioritize XComs and
other specific routes, ensuring they are correctly preserved and not
inadvertently overridden. Let me know your thoughts or if there are additional
areas we should consider reviewing!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
