• The field "preCanonicalized" which may cause infinite recursive calls is skipped during serialization.
• Modified the test to return truly valid JSON from the serialization method

Problem

1. LogicalPlanSerializer returns empty string in case of the Exception. Empty string is not a valid JSON and may render the whole event unparseable when used.
2. The field preCanonicalized can cause StackOverflowError because of infinite recursion

Solution

1. LogicalPlanSerializer returns "<failed-to-serialize-logical-plan>" for failed serialization instead of empty string.
2. The field preCanonicalized was excluded from serialization.

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

• Your change modifies the core OpenLineage model
• Your change modifies one or more OpenLineage facets

If you're contributing a new integration, please specify the scope of the integration and how/where it has been tested (e.g., Apache Spark integration supports S3 and GCS filesystem operations, tested with AWS EMR).

One-line summary:

Checklist

• You've signed-off your work
• Your pull request title follows our guidelines
• 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)
• Your comment includes a one-liner for the changelog about the specific purpose of the change (not required for changes to tests, docs, or CI config)
• You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
• You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2024 contributors to the OpenLineage project