validate RunLink and JobLink

[pawel-big-lebowski]

Signed-off-by: Pawel Leszczynski leszczynski.pawel@gmail.com

Problem

ParentRunFacet may contain empty run section which results in NullPointerException.

Closes: #2263

Solution

Fix validation of ParentRunFacet

Note: All database schema changes require discussion. Please link the issue for context.

Checklist

• You've signed-off your work
• Your changes are accompanied by tests (if relevant)
• Your change contains a small diff and is self-contained
• You've updated any relevant documentation (if relevant)
• You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
• You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)
• You've included a header in any source code files (if relevant)

[codecov]

Codecov Report

Merging #2342 (83f9bf5) into main (ae6250a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #2342   +/-   ##
=========================================
  Coverage     76.72%   76.72%           
  Complexity     1177     1177           
=========================================
  Files           222      222           
  Lines          5354     5354           
  Branches        429      429           
=========================================
  Hits           4108     4108           
  Misses          768      768           
  Partials        478      478           

Impacted Files	Coverage Δ
...main/java/marquez/service/models/LineageEvent.java	86.30% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[wslulciuc]

Looks good! Given that the ParentRunFacet marks the run and job as required, it makes sense to enforce the contract.