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.

Sign up for GitHub

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

Jump to bottom

Use JSON partial updates #111

Open
marein opened this issue Apr 1, 2022 · 1 comment
Open

Use JSON partial updates #111

marein opened this issue Apr 1, 2022 · 1 comment
Labels
domain:connect-four priority:low type:performance

Comments

@marein
Copy link
Owner

marein commented Apr 1, 2022

MySQL is used as a document store in the connect four context. Currently the aggregate column gets the whole serialized game when changes are made, see here. This can be improved by partially updating the JSON column which is available since MySQL version 8.0, see here.
@marein
Copy link
Owner Author

marein commented Apr 11, 2024

Tested this a few months ago. No notable difference in performance. Would only be interesting for replication or reducing network traffic at the moment. The added code complexity doesn't justify this, as neither of the two benefits are currently bottlenecks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
domain:connect-four priority:low type:performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@marein