Texera / texera

Collaborative Machine-Learning-Centric Data Analytics Using Workflows
https://texera.github.io
Apache License 2.0
161 stars 68 forks source link

Remove userId from WorkflowContext #2800

Closed bobbai00 closed 3 weeks ago

bobbai00 commented 3 weeks ago

This PR removes userId from WorkflowContext.

Purpose of the removal

userId requires the WorkflowContext to have the session info. This is not clean as some components like WorkflowCompiler should not have dependencies with user session.

Usages of userId

There are only few usages related to WorkflowContext.userId. Most of these usages are redundant.

As userId was used for resolving user files, which is replaced by dataset now. It does not depend on or cause dependency of session.