justinpakzad commented on code in PR #70103:
URL: https://github.com/apache/airflow/pull/70103#discussion_r3890928483


##########
providers/snowflake/src/airflow/providers/snowflake/hooks/snowflake.py:
##########
@@ -64,6 +65,39 @@
     from airflow.providers.openlineage.sqlparser import DatabaseInfo
 
 
+class SnowparkContainerJobStatus(str, Enum):

Review Comment:
   While I was working on a follow up to implement retries (so a transient 
failure doesn't terminate the trigger/operator), I realized the SPCS trigger 
and operator will need some shared helpers beyond the statuses. So instead of 
putting them in the hook, I've moved them into a util new file in the existing 
`/utils` directory. This avoids the need to alias the status imports to 
maintain backwards compatibility and enables adding new helpers that are only 
used for SPCS.



-- 
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]

Reply via email to