uranusjr commented on PR #36075: URL: https://github.com/apache/airflow/pull/36075#issuecomment-1842180077
Do we need to do the merge at all? From what I can tell, the `extra` parameter on `register_dataset_change` is currently not used anywhere, and I suspect the original intention was to use that field to forward extra from the dataset directly to dataset event instead. Maybe we should just remove `extra=None` from `register_dataset_change` and just do the forwarding. cc @blag for some history on this -- 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]
