Skip to content
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

Remove the <Z> from JmsMatsFactory #50

Open
stolsvik opened this issue Feb 7, 2022 · 0 comments
Open

Remove the <Z> from JmsMatsFactory #50

stolsvik opened this issue Feb 7, 2022 · 0 comments
Labels
improvement Improve an existing feature

Comments

@stolsvik
Copy link
Contributor

stolsvik commented Feb 7, 2022

The type here is just a "transfer type" between the MatsTrace and the MatsSerializer, which are intrinsically linked, and should not be exposed to the user.

Object is probably the better type here: JmsMatsFactory just passes an Object (serialized STO or DTO) from the MatsTrace to the MatsSerializer, and it will have to cast it to whatever it really is.

Another option is possibly a "Holder" - but I currently don't see how that would work with the MatsTraceFieldImpl, which needs an actual type for the STO and DTO serialized fields.

@stolsvik stolsvik added the improvement Improve an existing feature label Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improve an existing feature
Projects
None yet
Development

No branches or pull requests

1 participant