hansva commented on PR #8017: URL: https://github.com/apache/hop/pull/8017#issuecomment-5341611846
I'll investigate a bit further, but I think we are both somewhat on the same path here. There will be a registry with rules that are being applied to do bidirectional type mapping, and external plugins can add their own mappings. The only difference I see here is that what's currently in this PR is at a global level, and the `TargetTypeMappingRule` in your code is at the connection level, which is a nice user-level override. I'll make sure the code keeps playing along nicely, we should be moving forward not backwards with these types of changes -- 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]
