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

versatile-data-kit: Meta Job example #1640

Merged
merged 6 commits into from
Feb 17, 2023

Conversation

gageorgiev
Copy link
Contributor

This PR introduces a wiki example demonstrating how a user could implement a Meta Job and component jobs.

Signed-off-by: Gabriel Georgiev gageorgiev@vmware.com

This PR introduces a wiki example demonstrating how
a user could implement a Meta Job and component jobs.

Signed-off-by: Gabriel Georgiev <gageorgiev@vmware.com>
Signed-off-by: Gabriel Georgiev <gageorgiev@vmware.com>
Signed-off-by: Gabriel Georgiev <gageorgiev@vmware.com>
@dvalkova
Copy link
Collaborator

I went through the README and the sample data jobs. I think the example is great, easy to follow, with no unnecessary text or explanations. The only thing I would add to the README is a short explanation about the MetaJobInput interface - just 1-2 sentences to stress on the fact that this is a different interface to the IJobInput we're used to using (or maybe add just a comment in the meta job script about that). This is not a requirement, you can decide whether to implement it.

Copy link
Collaborator

@dvalkova dvalkova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ship it :)

@gageorgiev gageorgiev enabled auto-merge (squash) February 17, 2023 17:15
@gageorgiev gageorgiev merged commit 7183366 into main Feb 17, 2023
@gageorgiev gageorgiev deleted the person/gageorgiev/metajob-example branch February 17, 2023 17:16
ivakoleva pushed a commit that referenced this pull request Feb 20, 2023
This PR introduces a wiki example demonstrating how a user could
implement a Meta Job and component jobs.

Signed-off-by: Gabriel Georgiev <gageorgiev@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants