Skip to content

We don't care about serialization verbosity #73

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

Closed
enikao opened this issue Feb 2, 2023 · 4 comments
Closed

We don't care about serialization verbosity #73

enikao opened this issue Feb 2, 2023 · 4 comments

Comments

@enikao
Copy link
Contributor

enikao commented Feb 2, 2023

Reducing the size of serialized models (and other data) is not a goal for LIonWeb.
We accept more verbose (aka. longer, aka bigger) serialized models in order to reduce complexity.
If size became an issue, we would tackle it on a different level, e.g. compression of transport format.

@enikao enikao added the repo label Feb 2, 2023
@sergej-koscejev
Copy link

sergej-koscejev commented Feb 2, 2023

Besides size, there's also human readability. It's easier to read a structure that only contains the important data, compared to a structure with a lot of boilerplate properties set to null values.

Related: the decision to Base64-encode concept IDs makes them non-human-readable in the JSON.

Would it be a problem if we would by default allow to leave out fields that are optional/null, unless there are reasons not to?

@enikao
Copy link
Contributor Author

enikao commented Feb 2, 2023

Related: the decision to Base64-encode concept IDs makes them non-human-readable in the JSON.

We discussed this as feature, not a bug in #31

@enikao
Copy link
Contributor Author

enikao commented Feb 2, 2023

Would it be a problem if we would by default allow to leave out fields that are optional/null, unless there are reasons not to?

Arguments at #59

@enikao
Copy link
Contributor Author

enikao commented Feb 17, 2023

In the meeting at 2023-02-17, we confirmed that we don't care about no stinkin' serialization verbosity.

@enikao enikao closed this as completed Feb 17, 2023
@enikao enikao added serialization and removed repo labels Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants