bugraoz93 commented on code in PR #45577:
URL: https://github.com/apache/airflow/pull/45577#discussion_r1912514278
##########
airflow/api_fastapi/core_api/routes/public/variables.py:
##########
@@ -235,3 +241,125 @@ def import_variables(
import_count=len(import_keys),
created_variable_keys=list(create_keys),
)
+
+
+def categorize_keys(session, keys: set) -> tuple[set, set]:
Review Comment:
I think we should start creating a service for these
non-controller(endpoint) code pieces and put them [under
here](https://github.com/apache/airflow/tree/main/airflow/api_fastapi/core_api/services)
similar to the UI case. I believe this wasn't needed before
--
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]
