ColtenOuO commented on PR #71203: URL: https://github.com/apache/airflow/pull/71203#issuecomment-5250621235
It looks like we might run into an issue using a set here. Since SerializedBaseOperator has `__hash__ = None` (and `__eq__` returns NotImplemented), the objects themselves are unhashable and can't be stored in a set. As a workaround, we could take a different approach: rely on `matched_task_ids: set[str]` as the primary data structure and skip creating the matched_tasks (list of objects) altogether. We'd just look up the object properties only when needed (ex. `self.task_dict[tid]`). I've changed the code to this approach. Let me know if this is what you were looking for >< thanks again! -- 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]
