dstandish commented on code in PR #40894:
URL: https://github.com/apache/airflow/pull/40894#discussion_r1684979063
##########
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:
i think the reason i did not do this before is because when you hit this,
item will be the thing on DAG requested, not `dag` for example.
e.g. it would fire if you did
`ti.task.dag.task_dict` and the error would be like
```
Attribute was removed on serialization and must be set again: task_dict
```
but the thing that was actually removed is `dag`
not that this isn't a fine thing to do but that was the thought
--
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]
