Skip to content

[Feature Request]: Handling columns with different meta types than String in Row Normalizer #3636

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
vdwals opened this issue Feb 16, 2024 · 2 comments
Assignees
Labels
P2 Default Priority Transforms

Comments

@vdwals
Copy link
Contributor

vdwals commented Feb 16, 2024

What would you like to happen?

In the context of #2312 and #3633, an issue has been identified regarding the normalization of rows with different meta types. It's unclear if this affects only non-String meta types or any mix of types.

Currently, the value of any type is simply copied without casting or conversion into the result rows, but its field receives the meta type String. This causes some functionalities to fail:

Potential solutions include adapting the Row Normalizer Transformation or aligning the documentation. Proposed solutions are:

  • Allowing users via the UI to specify format masks and other transformation-related details, enabling the Row Normalizer to transform non-String values.
  • Restricting users from selecting non-String fields in the UI.
  • Introducing a flag to cast/convert everything to String during the process.
  • Adding a dropdown to select the output field type, with String as the default for backward compatibility.

Further discussion is encouraged to determine the optimal solution.

Issue Priority

Priority: 2

Issue Component

Component: Transforms

@vdwals
Copy link
Contributor Author

vdwals commented Feb 16, 2024

This examples shows the error message appearing in the logs mentioned in #2312:
feature-request-3636.zip

@vdwals
Copy link
Contributor Author

vdwals commented Feb 16, 2024

.take-issue

vdwals added a commit to vdwals/hop that referenced this issue Feb 19, 2024
Updated documentation of Row normaliser to avoid issues stated in apache#3636 .
vdwals added a commit to vdwals/hop that referenced this issue Feb 19, 2024
Updated documentation to avoid issues in relation to apache#3636, where fields of different meta types are put into the same result field leading to potential issues with subsequent transforms.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Default Priority Transforms
Projects
None yet
Development

No branches or pull requests

2 participants