Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

feat: data stored as json #112

Merged
merged 7 commits into from
Feb 28, 2024
Merged

feat: data stored as json #112

merged 7 commits into from
Feb 28, 2024

Conversation

rllola
Copy link
Contributor

@rllola rllola commented Feb 9, 2024

Saving the data in JSON would allow to store all the transaction data in one table making indexing faster but looking for data maybe a little bit slower. It also has the advantage of removing a lot of code.

This need to be tested before merged.

Copy link
Collaborator

@neithanmo neithanmo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me, this reduces database layout complexity, and there is no need for more tables for each transaction type. as you said, we would need to benchmark this, and it will depend on query complexity, for example, one that returns a list of all transfer transactions linked to a specific target, source or key. this will for sure take more time to process whereas having a tx_transfer table would be faster?, again this would depend on query complexity which would need to process that JSON.

@rllola rllola marked this pull request as ready for review February 26, 2024 11:19
@rllola rllola mentioned this pull request Feb 28, 2024
@rllola rllola changed the title [WIP] feat: data stored as json feat: data stored as json Feb 28, 2024
@rllola rllola merged commit 8fc06f0 into main Feb 28, 2024
6 checks passed
@rllola rllola deleted the feat/json-sql branch February 28, 2024 15:23
@joel-u410 joel-u410 mentioned this pull request Mar 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants