Skip to content

fix #3056#3057

Closed
sramazzina wants to merge 3 commits intoapache:masterfrom
sramazzina:NULL-POINTER
Closed

fix #3056#3057
sramazzina wants to merge 3 commits intoapache:masterfrom
sramazzina:NULL-POINTER

Conversation

@sramazzina
Copy link
Copy Markdown
Contributor

@sramazzina sramazzina commented Jul 6, 2023

fix #3056 NullPointerException on pipelines with transforms receiving more than one flow in input

The following considerations applies to this PR:

  1. This fix start from the assumption when we arrive to a transform, we arrive there because we have a flow that takes us there otherwise we never arrive there.
  2. We definitely verified that the problem arises because, at the time the issue is raised, the inputRowMeta reference is null
  3. The place where this change acts is the only place where, in the engine, the inputRowMeta reference is set during the lifecycle of the transform
  4. If we have more than one flow that is entering a transform, the change keeps only the RowMeta references that are valid and we suppose that at least one of these must be valid because of the assumption made at point 1 above

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
  • If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
  • Mention the appropriate issue in your description (for example: addresses #123), if applicable.

To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

@sramazzina sramazzina closed this Sep 29, 2023
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.

[Bug]: NullPointerException on pipelines with transforms receiving more than one flow in input

1 participant