-
Notifications
You must be signed in to change notification settings - Fork 145
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PutContexts failed: Given node already exists #52
Comments
hi @calvinleungyk , this is WAI. in mlmd, the context {name, type} is unique [1]. In this case, this context has been created five times. First time it will succeed, the rest of it, will return an error. In the tfx runtime, this is ignored and log errors to the users, and continue. |
Thanks for the response @hughmiao. Is it something I've done in my pipeline that it's created 5 times, or is it something done in a default way in TFX? I don't remember doing any creation other than the one InteractiveContext |
I believe this is handled by tfx and the interactive context implementation. Let me cc some tfx folks to take a look. @ruoyu90 @charlesccychen |
@calvinleungyk this is expected. Basically every node you run will try to register the pipeline context and reuse the one if exists. This is the reason you constantly see four warnings. |
thanks @ruoyu90 to confirm. |
Hi team, I ran a simple pipeline on JupyterLab that consists of InteractiveContext, ExampleGen, StatisticsGen, SchemaGen, ExampleValidator and Trainer but ml-metadata gRPC server keeps outputting this warning:
There are 5 components and this warning appears 4 times in a row and each time the name is identical.
My understanding is
pipeline_name
is used in both Artifacts and Contexts and it should allow update on a Context with existing name. if name already exists, it would be an update, but I'm not sure why put_contexts is repeatedly called. Has anyone seen this behavior / is this behavior expected or is something wrong?Thank you.
The text was updated successfully, but these errors were encountered: