tolerate null transformation attrs in ColumnLineageInputField

[davidjgoss]

Problem

The transformationType and transformationDescription attributes for column-level lineage are optional at the database level and can be safely omitted when pushing in lineage events.

However, because they're marked as non-nullable in the Java client's representation, an exception is thrown when deserialising a response from the column lineage endpoint (thanks to Lombok-generated Objects.requireNonNull in the constructor).

Solution

Remove the @NonNull annotation from the client class and the @NotNull from the model class. Add a test that would fail without this change. Validated by running the stack locally.

One-line summary:
tolerate null transformation attrs in field model

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 included a one-line summary of your change for the CHANGELOG.md (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 #2600 (2ddca22) into main (e48ac69) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #2600   +/-   ##
=========================================
  Coverage     83.30%   83.30%           
  Complexity     1287     1287           
=========================================
  Files           243      243           
  Lines          5935     5935           
  Branches        279      279           
=========================================
  Hits           4944     4944           
  Misses          844      844           
  Partials        147      147           

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

[pawel-big-lebowski]

Indeed, transformation attrs are not required within the spec -> https://github.com/OpenLineage/OpenLineage/blob/main/spec/facets/ColumnLineageDatasetFacet.json#L51
so they shouldn't be obligatory in Marquez.

[pawel-big-lebowski]

Looks good to me!

[wslulciuc]

LGTM 👍