You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hop.apache.org by "sramazzina (via GitHub)" <gi...@apache.org> on 2023/07/06 13:43:10 UTC

[PR] fix #3056 (hop)

sramazzina opened a new pull request, #3057:
URL: https://github.com/apache/hop/pull/3057

   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](http://www.apache.org/licenses/LICENSE-2.0)
   you have to acknowledge this by using the following check-box.
   
   - [ ] I hereby declare this contribution to be licensed under the [Apache License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0)
   - [ ] In any other case, please file an [Apache Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix #3056 (hop)

Posted by "sramazzina (via GitHub)" <gi...@apache.org>.
sramazzina closed pull request #3057: fix #3056
URL: https://github.com/apache/hop/pull/3057


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org