dstandish commented on code in PR #40894:
URL: https://github.com/apache/airflow/pull/40894#discussion_r1685020469
##########
airflow/utils/types.py:
##########
@@ -54,7 +54,7 @@ class AttributeRemoved:
"""
def __getattr__(self, item):
- raise RuntimeError("Attribute was removed on serialization and must be
set again.")
+ raise RuntimeError(f"Attribute was removed on serialization and must
be set again: {item}.")
Review Comment:
Yeah it's not completely meaningless. You would have this statement
`Attribute was removed on serialization and must be set again` and then to
answer the question "which attribute" you have the traceback. Referencing the
wrong attr might be worse than not referencing any. That's where i landed
anyway. But no objection from me if you think this is better.
--
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]
