rmannibucau opened a new pull request, #8813:
URL: https://github.com/apache/hop/pull/8813
fixes #8812
The HopMetadataInjector rewrite (4c51db7edc) copies values from injection
row buffers into String fields and List<String> fields as-is. When the source
is a lazy-conversion transform (e.g. CSV Input with lazy conversion enabled),
those values are raw byte[] and end up injected as their toString()
representation ("[B@..."), which later breaks SimpleDateFormat with "Illegal
pattern character 'B'" when the value is used as a format mask (e.g. Calculator
conversion_mask).
Decode values with STORAGE_TYPE_BINARY_STRING to their native type via
convertBinaryStringToNativeType() in both injectList() branches, matching the
legacy BeanInjector path which already decoded through
RowMetaAndData.getString()/getAsJavaType().
Adds regression tests for List<String> injection and String fields of list
items fed from lazy byte[] row buffers.
**Please** add a meaningful description for your change here
------------------------
Thank you for your contribution! Follow this checklist to help us
incorporate your contribution quickly and easily:
- [X] 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.
- [X] 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`.
- [X] 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.
- [X] 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)
- [X] 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
