23tae commented on code in PR #68890:
URL: https://github.com/apache/airflow/pull/68890#discussion_r3682985095
##########
airflow-core/src/airflow/utils/helpers.py:
##########
@@ -55,16 +55,14 @@ def validate_key(k: str, max_length: int = 250):
if not isinstance(k, str):
raise TypeError(f"The key has to be a string and is {type(k)}:{k}")
if len(k) > max_length:
- raise AirflowException(f"The key: {k} has to be less than {max_length}
characters")
+ raise ValueError(f"The key: {k} has to be less than {max_length}
characters")
Review Comment:
Thanks for the review. Makes sense. I'll add the newsfragment for the KPO
impact and align the error message with the SDK.
--
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]
