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

API: allow constructing pipeliner builders from pre-existing pipeline builders #25

Open
eledhwen opened this issue Mar 6, 2024 · 2 comments
Labels

Comments

@eledhwen
Copy link
Collaborator

eledhwen commented Mar 6, 2024

The is currently no way to derive a pipeline builder from an existing pipeline builder instance.

This type of feature is useful for situations such as having a Spring bean for a "base" pipeline builder instance, which other beans would fork from, tweak, and then build into their respective pipeline bean. We can already have something similar by creating "factory" functions instead, but in this case we need to explicitly wire all required arguments (which can introduce discrepancies).

The general idea is along the lines of:

var newBuilder = Pipeline.of(existingBuilder);
@nitintecg
Copy link

@eledhwen I would like to contribute on this issue but having one doubt below :
Are steps and sinks fields necessary only for creating a pipeline builder from an existing builder?
Thanks

@eledhwen
Copy link
Collaborator Author

eledhwen commented Mar 8, 2024

Hi there.

Thank you for chiming in! I'm not sure I understand the question, do you use the library in some capacity?

It might be appropriate to familiarize yourself with what it does (and how) before diving in potential contributions.

This is a "good first issue" as long as one has some grasp on what is actually being attempted.

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

No branches or pull requests

2 participants