Skip to content
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

[feature] [SDK] Extract Executor to its own package #11345

Open
leseb opened this issue Oct 31, 2024 · 6 comments
Open

[feature] [SDK] Extract Executor to its own package #11345

leseb opened this issue Oct 31, 2024 · 6 comments

Comments

@leseb
Copy link
Contributor

leseb commented Oct 31, 2024

Feature Area

/area sdk

What feature would you like to see?

To improve modularity and reduce dependency overhead in Kubeflow Pipelines, I propose extracting the executor component into a standalone package. Currently, the executor is tightly coupled with other SDK components, resulting in unnecessary dependencies during execution. By isolating the executor into its own package, we can minimize the dependency footprint and enhance flexibility for users seeking a more lightweight solution.

Something like kfp.executor package would be nice. Thanks for considering this proposal.

Love this idea? Give it a 👍.

@leseb
Copy link
Contributor Author

leseb commented Oct 31, 2024

@HumairAK FYI

@HumairAK
Copy link
Collaborator

HumairAK commented Nov 6, 2024

past attempt on this: #9710

@chensun
Copy link
Member

chensun commented Nov 6, 2024

Something like kfp.executor package would be nice. Thanks for considering this proposal.

Second part of this effort was #9738, which was later reverted by #9874 due to some technical challenges IIRC. @connor-mccarthy do you recall the exact issue?

@HumairAK
Copy link
Collaborator

HumairAK commented Nov 6, 2024

In today's community call it was decided that this approach makes sense and got a +1 to move forward.

@chensun
Copy link
Member

chensun commented Nov 6, 2024

And what we ended up as a shortcut was to install kfp without any dependencies:

https://github.com/connor-mccarthy/pipelines/blob/fcdff294a6323f6cb1c0e574fc7aa5ccc25e420b/sdk/python/kfp/dsl/component_factory.py#L146-L149

which is arguably the same level of dependency as installing a subpackage like kfp.runtime.

@leseb
Copy link
Contributor Author

leseb commented Nov 7, 2024

Could we get a list of the technical challenges that caused the initial revert of the previous attempt? This will help us avoid repeating the same issues. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants