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

Data availability #124

Open
wants to merge 210 commits into
base: main
Choose a base branch
from
Open

Data availability #124

wants to merge 210 commits into from

Conversation

Rom3dius
Copy link
Contributor

Stores withdrawals, transfers and deposits in a single postgresql table. Allows to query for a maximum amount of 500 transactions, using the following filters:

  • sender
  • recipient
  • min amount
  • max amount
  • min timestamp
  • max timestamp

Timestamp is created right before insert.

Testing needs work, tests currently fail because test setup is synchronous but pool setup needs async runtime. And of course the missing database. This is the current testing connection string: postgres://kairos:kairos@localhost/kairos

koxu1996 and others added 30 commits May 10, 2024 14:45
@koxu1996
Add casper-event-toolkit dependency.
27f2c82
@koxu1996
Add reqwest dependency.
b9f2713
@koxu1996
Add thiserror dependency.
fbf99e5
@koxu1996
Prototype of state synchronization.
03544d2
@koxu1996
Fix deadlock.
a60cbb9
@koxu1996
Use amount greater than 0 for mocked tx.
6897e36
@koxu1996
Merge remote-tracking branch 'origin/feature/configure-casper-rpc-in-…
413c599 
…server' into feature/l1-synchronization
@koxu1996
Move L1 sync into separate function.
8806f1b
@koxu1996
Skip synchronization if contract hash not configured.
a4b0434
@koxu1996
Use scoped ENV for contract hash.
88aa500
@koxu1996
Merge branch 'main' into feature/l1-synchronization
70c7057
@koxu1996
Fix default contract hash value.
f57b840
@koxu1996
Add default contract hash to nix setup.
308577b
@marijanp
flake: add ci binary cache
a26e9c3
@jonas089
wip - trie
7e198ff
@jonas089
wip - nix
cd89d07
@Avi-D-coder
Add kairos-prover/kairos-circuit-logic to kairos-contracts nix sources
bacd8ee
@jonas089
enable risc0 std to fix build
a9dc562
@jonas089
hacky workaround - use risc0 std handler in malicious sessions instea…
a8c5de5 
…d of casper-contract no-std-helper
@jonas089
use panic handler from casper-contract std instead
7b6b721
@jonas089
fix std deps and contract wasm size
b2d205b
@jonas089
use no-std-helpers
452e926
@koxu1996
Bump casper-event-toolkit to v0.1.3.
c2ccfdf
@koxu1996
Remove into() that is no longer required.
c09d3fd
@koxu1996
Replace unused Result match with map_err.
1fac715
@koxu1996
Replace another unused Result with map_err.
be86df8
@koxu1996
Replace yet another unused Result with map_err.
326b147
@koxu1996
Get rid of Mutex for EventManager.
15a1197
@koxu1996
Return error if L1 processing started without proper init.
1f1b1d3
@koxu1996
Prepare sync service for storing event manager handle.
1065394
koxu1996
koxu1996 approved these changes Jun 24, 2024
View reviewed changes
Copy link
Contributor

@koxu1996 koxu1996 left a comment

Choose a reason for hiding this comment

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

LGTM. One remark: I think we should store transactions in kairos-tx format.

@jonas089 jonas089 self-requested a review June 24, 2024 11:47
jonas089
jonas089 approved these changes Jun 24, 2024
View reviewed changes
Copy link
Contributor

@jonas089 jonas089 left a comment

Choose a reason for hiding this comment

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

lgtm

marijanp and others added 22 commits June 24, 2024 15:48
@marijanp
Merge remote-tracking branch 'origin/main' into nixos-add-chainspec
fba3c53
@marijanp
tests/e2e: revert back to use the same chainspec as in contract tests
358b8bd
@marijanp
kairos-contracts/contract-utils: remove unused casper-types dependency
3e96548
@marijanp
kairos-contracts/demo-contract: use u32 for the last processed deposi…
32af194 
…t counter
@marijanp
kairos-test-utils/cctl: add deploy contract function
7755159
@marijanp
Merge pull request #125 from cspr-rad/nixos-add-chainspec
4b57964 
nixos/cctl: use exponential backoff, remove unused code
@marijanp
Merge remote-tracking branch 'origin/main' into add-deploy-contract-cctl
c35e2ed
@marijanp
Merge branch 'main' into fixup-deposit-counter-type
5c705e3
@marijanp
Merge pull request #139 from cspr-rad/fixup-deposit-counter-type
37186e1
@marijanp
Merge branch 'main' into add-deploy-contract-cctl
af899f9
@Avi-D-coder
Merge pull request #136 from cspr-rad/add-deploy-contract-cctl
341dba0 
kairos-test-utils/cctl: add deploy contract function
@Rom3dius
store transfer, withdrawal deposit + query endpoint
904435b
@Rom3dius
implemented From<DBError> for AppErr
30ccdc7
@Rom3dius
put it all behind a feature flag
3dc0ca7
@Rom3dius
use bigdecimal for numeric + only run migrations if feature enabled
1d15b09
@Rom3dius
added migrations feature to kairos-server kairos-data
c1980dc
@Rom3dius
enum + fmt + removed comments + removed blankspace etc.
b4f4bb4
@Rom3dius
removed all pool clones
eb3dced
@Rom3dius
rebased + fixes
0b4ed49
@Rom3dius
database flag for test-utils + feature flag propagation
9067e1d
@Rom3dius
Merge branch 'feature/data-layer' of github.com:cspr-rad/kairos into …
13fb76b 
…feature/data-layer
@Rom3dius
removed duplicate kairo-sdata
f3db973
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Avi-D-coder Avi-D-coder Avi-D-coder left review comments

@koxu1996 koxu1996 koxu1996 approved these changes

@marijanp marijanp marijanp approved these changes

@jonas089 jonas089 jonas089 approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

@Rom3dius Rom3dius

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@Rom3dius @koxu1996 @marijanp @Avi-D-coder @jonas089