Skip to content

Update run-lifecycle.md to add runId requirement on job finish - #863

Merged
julienledem merged 1 commit into
mainfrom
julienledem-add-runId
Dec 22, 2020
Merged

Update run-lifecycle.md to add runId requirement on job finish#863
julienledem merged 1 commit into
mainfrom
julienledem-add-runId

Conversation

@julienledem

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Dec 7, 2020

Copy link
Copy Markdown

Codecov Report

Merging #863 (444e031) into main (5e00aae) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #863   +/-   ##
=========================================
  Coverage     72.88%   72.88%           
  Complexity      437      437           
=========================================
  Files           112      112           
  Lines          2054     2054           
  Branches        188      188           
=========================================
  Hits           1497     1497           
  Misses          269      269           
  Partials        288      288           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 03052eb...3868ce8. Read the comment docs.

henneberger added a commit to henneberger/marquez-airflow that referenced this pull request Dec 8, 2020
…api sequence to conform to MarquezProject/marquez/pull/863

Signed-off-by: henneberger <git@danielhenneberger.com>
wslulciuc pushed a commit to MarquezProject/marquez-airflow that referenced this pull request Dec 8, 2020
)

* Refactor + Fix extract not reporting task state on callback + update api sequence to conform to MarquezProject/marquez/pull/863

Signed-off-by: henneberger <git@danielhenneberger.com>

@wslulciuc wslulciuc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 👍

@wslulciuc

Copy link
Copy Markdown
Member

@julienledem: Looks like you need to sign the commit before we can merge? :)

Signed-off-by: Julien Le Dem <julien@apache.org>
@julienledem
julienledem force-pushed the julienledem-add-runId branch from 444e031 to 3868ce8 Compare December 22, 2020 23:31
@julienledem

Copy link
Copy Markdown
Member Author

rebased and signed the commit.

@julienledem
julienledem merged commit 148c300 into main Dec 22, 2020
@julienledem
julienledem deleted the julienledem-add-runId branch December 22, 2020 23:31
hcthakur2004 pushed a commit to hcthakur2004/OpenLineage that referenced this pull request Jul 5, 2026
…penLineage#114)

* Refactor + Fix extract not reporting task state on callback + update api sequence to conform to MarquezProject/marquez/pull/863

Signed-off-by: henneberger <git@danielhenneberger.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants