Skip to content

Conversation

@loomlike
Copy link
Collaborator

Signed-off-by: Jun Ki Min [email protected]

Description

Add job_tag to materialization job submission.
Include output_path to job_tag when using an offline sink.

This PR fixes current CI test error.

How was this PR tested?

Does this PR introduce any user-facing changes?

  • No. You can skip the rest of this section.
  • Yes. Make sure to clarify your proposed changes.

… arg order to make it work with old codes

Signed-off-by: Jun Ki Min <[email protected]>
@loomlike loomlike added the safe to test Tag to execute build pipeline for a PR from forked repo label Nov 30, 2022
@xiaoyongzhu xiaoyongzhu merged commit 2d6da56 into feathr-ai:main Dec 1, 2022
jaymo001 pushed a commit that referenced this pull request Dec 7, 2022
… arg order to make it work with old codes (#890)

Signed-off-by: Jun Ki Min <[email protected]>

Signed-off-by: Jun Ki Min <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe to test Tag to execute build pipeline for a PR from forked repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants