kacpermuda commented on PR #51203: URL: https://github.com/apache/airflow/pull/51203#issuecomment-2948896800
It's implemented this way because when providing description to the EventsTimetable, both `_summary` and `description` attributes are set to the provided `description`. When not provided, the `_summary` and `description` are generated, and different from each other. So if we serialize `description` regardless if it was provided or not, after deserializing the resulting summary will be different than before serialization if the description was not provided by the user. We would end up displaying the description (longer text) in the Airflow UI if we did that. We can also add the summary argument explicitly, but that was a bigger change imho. -- 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]
