jscheffl commented on code in PR #43611:
URL: https://github.com/apache/airflow/pull/43611#discussion_r1841057314
##########
task_sdk/src/airflow/sdk/definitions/baseoperator.py:
##########
@@ -869,6 +874,11 @@ def __init__(
validate_instance_args(self, BASEOPERATOR_ARGS_EXPECTED_TYPES)
+ # Ensure priority_weight is within the valid range
+ self.priority_weight = max(
+ MINIMUM_PRIORITY_WEIGHT, min(MAXIMUM_PRIORITY_WEIGHT,
self.priority_weight)
+ )
Review Comment:
Would like to but the implementaiton currently is in `airflow-utils` - I
asusme we want to have task SDK split apart from core?
Still what I can do is implement the same in task-SDK as utility... the old
abstract/base operator in core will go away once AIP-72 is complete, correct?
--
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]
