tkobayas commented on issue #7201: URL: https://github.com/apache/incubator-kie/issues/7201#issuecomment-6261664478
# Sequential Mode Beta Node Sharing Test Results Test class: `SequentialNodeSharingTest.java` -> https://gist.github.com/tkobayas/a61725c1ebe636266c0f3c7da322a054 ## Summary - **Sharing OFF**: All tests pass with correct results. - **Sharing ON**: 4 NPEs (all left update) + 4 incorrect results (right update or left update without NPE). ## Results ### FromNode | Test | Sharing ON | Sharing OFF | Status | |---|---|---|---| | left update | **NPE**: `previousMatches` is null in `PhreakFromNode.doLeftUpdates` | `[R1, R2:London]` | BUG | | right update | `[R1, R2:Paris]` | `[R1, R2:Paris]` | OK | ### JoinNode | Test | Sharing ON | Sharing OFF | Status | |---|---|---|---| | left update | **NPE**: `ltm` is null in `doUpdatesReorderLeftMemory` | `[R1, R2:stilton]` | BUG | | right update | `[R1, R2:cheddar]` | `[R1]` | **Wrong result**: R2 fires with stale match | ### AccumulateNode | Test | Sharing ON | Sharing OFF | Status | |---|---|---|---| | left update | **NPE**: `ltm` is null in `doUpdatesReorderLeftMemory` | `[R1:30, R2:30]` | BUG | | right update | `[R1:10, R1b, R2:10]` | `[R1:10, R1b, R2:99]` | **Wrong result**: R2 sees pre-update value (10 instead of 99) | ### NotNode | Test | Sharing ON | Sharing OFF | Status | |---|---|---|---| | left update | **NPE**: `ltm` is null in `doUpdatesExistentialReorderLeftMemory` | `[R1, R2]` | BUG | | right update | `[R1]` | `[R1]` | OK | ### ExistsNode | Test | Sharing ON | Sharing OFF | Status | |---|---|---|---| | left update | `[R1, R2, R2]` | `[R1, R2]` | **Wrong result**: R2 fires twice | | right update | `[R1, R1b, R2]` | `[R1, R1b]` | **Wrong result**: R2 fires after stilton removed | ## NPE Locations | Node Type | Location | |---|---| | FromNode | `PhreakFromNode.doLeftUpdates` — `previousMatches` is null | | JoinNode | `PhreakNodeOperations.doUpdatesReorderLeftMemory` — `ltm` is null | | AccumulateNode | `PhreakNodeOperations.doUpdatesReorderLeftMemory` — `ltm` is null | | NotNode | `PhreakNodeOperations.doUpdatesExistentialReorderLeftMemory` — `ltm` is null | -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
