kacpermuda commented on PR #52441: URL: https://github.com/apache/airflow/pull/52441#issuecomment-3036263040
@kgw7401 @VladaZakharova I think that would the way to go, copying OL variable transport and operator to common test utils and then importing from there in google system tests. I can look into it maybe next week, so if you want to do it faster, I'll be happy to review the PR. Not sure how `import from openlineage.X` will behave in those common utils, so definitely something to check. Also @mobuchowski, do we want to move the OL VariableTransport and check operator into common utils entirely or also keep a copy in OL provider? Not sure it makes sense to keep a copy, but then any changes to it are nicely labeled with `provider:openlineage` and it helps me keep track of any changes made by other contributors, maybe there is a workaround for 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]
