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

[Proposal] New project to handle JSON serialization #509

Open
artursouza opened this issue Mar 11, 2021 · 3 comments
Open

[Proposal] New project to handle JSON serialization #509

artursouza opened this issue Mar 11, 2021 · 3 comments
Labels

Comments

@artursouza
Copy link
Member

artursouza commented Mar 11, 2021

The core SDK does not provide solutions for serialization of application domain objects, it only offers an interface. In order to enable applications to opt-in to specific serialization solutions, it might be better to expose as an optional project (similar to how it is done in Spring Boot).

This proposal is to create a new project for JSON serialization that apps can opt in and be extensible, so it can be configured to handle specific scenarios unique for each app. This package should be specific per vendor, which means a different JSON serializer should be done in a separate project.

This new project would also not be opinionated (but customizable) in how data types are serialized. Like DateTime, for example.

We can start with dapr-jackson project, where it uses the Jackson serializer. Possible names:

  • dapr-jackson
  • dapr-serialization-jackson
  • dapr-serialize-jackson

Open questions

  • How would apps implement their own versioning logic on top of this serializer?
@abogdanov37
Copy link
Contributor

It's an interesting idea. But I think that it should be a group of projects. For example, a common project dapr-serialization with common modules. And then dapr-serialization-jackson, dapr-serialization-jsoniter and other.

@artursouza
Copy link
Member Author

@abogdanov37 What would be in dapr-serialization? I wonder if it is something already present in dapr-sdk.

@stliu
Copy link

stliu commented Dec 7, 2021

totally agree, we have run into many dependencies issues caused by the hard dependency of jackson, you can find more here https://github.com/Azure/azure-sdk-for-java/issues?q=is%3Aissue+jackson, and azure sdk is planning to remove it Azure/azure-sdk-for-java#24480

meanwhile, you may also consider this Azure/azure-sdk-for-java#24530

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants