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

Add created_at timestamp to transaction #134

Merged
merged 1 commit into from
Mar 11, 2024
Merged

Add created_at timestamp to transaction #134

merged 1 commit into from
Mar 11, 2024

Conversation

tuommaki
Copy link
Contributor

Transaction creation timestamp can be used for categorizing historical transactions in to an immutable set that can be then used for syncing.

In Gevulot devnet, since there is no consensus, there is also no reliable way to order transactions correctly in a distributed system.

However, when each transaction still has a creation timestamp, that can be used as a snapshotting point where all historical transactions can be ordered by their hash and therefore grouped for quicker syncing.

Transaction creation timestamp can be used for categorizing historical
transactions in to an immutable set that can be then used for syncing.

In Gevulot devnet, since there is no consensus, there is also no
reliable way to order transactions correctly in a distributed system.

However, when each transaction still has a creation timestamp, that
can be used as a snapshotting point where all historical transactions
can be ordered by their hash and therefore grouped for quicker syncing.
@tuommaki tuommaki self-assigned this Mar 11, 2024
@tuommaki
Copy link
Contributor Author

😠 I just found out I've been sitting on this over a month. I thought it was already in main. Wondered where the **** the created_at field is from transaction table 🤦‍♂️

@tuommaki tuommaki requested a review from musitdev March 11, 2024 12:52
@tuommaki tuommaki merged commit 8bb4be9 into main Mar 11, 2024
4 checks passed
@tuommaki tuommaki deleted the add-tx-timestmap branch March 11, 2024 13:14
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