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

mina funnel #343

Merged
merged 31 commits into from
May 3, 2024
Merged

mina funnel #343

Show file tree
Hide file tree
Changes from 26 commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
d6cf581
mina funnel: wip
ecioppettini Mar 21, 2024
380eccc
wip
ecioppettini Apr 1, 2024
c6110ee
add actions fetching
ecioppettini Apr 4, 2024
09d5ad2
cleanup some lints
ecioppettini Apr 4, 2024
f6e8914
split actions and events into 2 cdes
ecioppettini Apr 9, 2024
7fc458c
add after/cursor argument to the queries
ecioppettini Apr 10, 2024
cba5de1
remove some debugging logs
ecioppettini Apr 10, 2024
ed6fa32
rename cardano cursors table to generic cursor based pagination
ecioppettini Apr 11, 2024
d3a63e9
add paginationLimit setting
ecioppettini Apr 11, 2024
6a18c16
dehardcode slot duration
ecioppettini Apr 11, 2024
ccdc0fc
replace the node's graphql api with direct db queries
ecioppettini Apr 13, 2024
1743988
replace the archive's graphql with direct db queries
ecioppettini Apr 13, 2024
a6f9155
add limit to the query
ecioppettini Apr 15, 2024
5a08d13
add loop to wait for finality
ecioppettini Apr 15, 2024
57af769
remove debugging logs
ecioppettini Apr 15, 2024
92e7e59
minor refactors
ecioppettini Apr 16, 2024
c1c0f46
remove unused fields from query
ecioppettini Apr 16, 2024
19e36a6
handle edge case for the cde generic handler
ecioppettini Apr 16, 2024
4c70366
fix upper range querying extra data
ecioppettini Apr 16, 2024
c8f2f96
find confirmed timestamp directly instead of getting the slot first
ecioppettini Apr 17, 2024
5060bca
rename the common paginated handling cde function
ecioppettini Apr 17, 2024
02aaa39
small refactor in funnel cursors filtering
ecioppettini Apr 18, 2024
6d54ea1
rework the timestamp to block mapping to not use slots
ecioppettini Apr 18, 2024
5e3ce4f
replace startSlot for startBlockHeight in the extensions
ecioppettini Apr 18, 2024
f62b086
add timestamp checkpointing event
ecioppettini Apr 23, 2024
88acf1c
genesisTime field is not used anymore
ecioppettini Apr 23, 2024
3ef89e8
fix missing cache update of the lower bound
ecioppettini Apr 27, 2024
24546d0
allow overriding archive's confirmation parameters
ecioppettini Apr 29, 2024
27dbb08
migrated from postgres to pg
ecioppettini Apr 30, 2024
26e543e
remove debugging logs
ecioppettini Apr 30, 2024
9c65d86
cleanup
ecioppettini May 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 14 additions & 1 deletion package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 2 additions & 1 deletion packages/engine/paima-funnel/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
"dependencies": {
"assert-never": "^1.2.1",
"@dcspark/carp-client": "^3.1.0",
"@dcspark/cardano-multiplatform-lib-nodejs": "5.2.0"
"@dcspark/cardano-multiplatform-lib-nodejs": "5.2.0",
"postgres": "^3.3.5"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why can't we use the pg / pg-tx libraries we already use?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

To be honest I just didn't think about it. I started with this dep because it was being used in the archive node api, and since I took that as the base it was simpler to get started. Then I was considering porting to pgtyped eventually, but I didn't think about that. Switching to pg should be simple, I'll do that first.

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah I think switching to pgtyped is a lower priority. If we can at least use pg to avoid having to depend on the postgres package as well I think that's good enough

}
}
Loading