uranusjr commented on issue #26499: URL: https://github.com/apache/airflow/issues/26499#issuecomment-1254604913
Ah, I missed the `self.length` part you changed in `__init__`. That could be the solution (if the identity is indeed the issue), although I’d prefer the logic to live in `__len__` instead of `__init__`. But if serialisation is the issue (and again, I am not yet entirely convinced that’s the case), the pre-commit checks and unit tests would not be enough to verify the solution. The easiest way to check would be for someone to actually run a DAG against this patch. -- 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]
