amoghrajesh commented on PR #56187: URL: https://github.com/apache/airflow/pull/56187#issuecomment-3540393912
@xBis7 I am already working on the shared config part and seems it is close now. Might take a little more time till I get done with that. About exceptions, which are the ones you mention? The idea of thumb is, if its only going to be within the shared library, move it to shared. If it is publicly exposed and used in core OR sdk, it has to be in either one of them, not both. If its going to be used in some form for dag authoring, custom providers, plugins etc, it would be better off in task sdk. I checked: - InvalidStatsNameException - looks like its only ever going to be in metrics, so it could live in shared metrics I couldnt find any other exception in metric / traces. -- 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]
