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

Denormalize Album class and embed it to Track entity #23

Closed
AddictedCS opened this issue Oct 5, 2013 · 1 comment
Closed

Denormalize Album class and embed it to Track entity #23

AddictedCS opened this issue Oct 5, 2013 · 1 comment
Assignees
Milestone

Comments

@AddictedCS
Copy link
Owner

Album model can be safely embedded into Track as it represents mainly read only information (it will be updated only in case if an error is introduced, which is unlikely). Having a typical read only model, would suggest denormalizing it into Track:

  • Album name
  • Album release year (is it really useful?)

Mark those fields as nullable (in case when album information is not known).

@ghost ghost assigned AddictedCS Oct 5, 2013
@AddictedCS
Copy link
Owner Author

Done.

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

No branches or pull requests

1 participant