S3tuit commented on issue #5700: URL: https://github.com/apache/hop/issues/5700#issuecomment-3324305202
I think this is a schema-race at `Sort rows result`. It has two inputs: `Memory group by` (5 fields) and `Group by` (6 fields, includes `NUMBERS_CRLF`). The unit-test row collector captures the schema of the first row for that target transform. If the first row comes from the 5-field branch, the collected `RowCollection` only has 5 `ValueMeta`, so `validateTransformResultAgainstUnitTest` can’t find `NUMBERS_CRLF` and fails. If the first row arrives from the 6-field branch, the test passes. Maybe the pipeline should add one of these: - `Stream Schema Merge` before `Sort rows result` or - add the missing 6th field to the `Memory group by`. But then we should change `NUMBERS_CRLF` since `Memory group by` doesn't have a `Concatenate strings separated by new line (CRLF)` option like `Group by` (should be added?). Personally, I like the first approach more since it's fater. If it sounds good, I can update the sample test pipeline and open a PR. -- 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]
