-
Notifications
You must be signed in to change notification settings - Fork 6
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.
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
Add example postgres processor #113
base: 03-04-postgres_crate
Are you sure you want to change the base?
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
944ff81
to
dfe9e00
Compare
29a5768
to
43228c9
Compare
a3b0e55
to
af1aa2d
Compare
43228c9
to
f2b3903
Compare
"libpq", | ||
] } | ||
anyhow = "1.0.86" | ||
aptos-indexer-processor-sdk-server-framework = { path = "../aptos-indexer-processors-sdk/sdk-server-framework" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: is this something that we can expose through sdk as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if this doesn't require a lot of additional imports we can just move this into the sdk without having an additional feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rtso did we decide to do this? if not in the vanilla crate maybe just in the postgres feature? otherwise i think its an unnecessary exposure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah we should consolidate crates. I'll follow up in another PR.
) | ||
.await; | ||
|
||
check_or_update_chain_id( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: should we mark/comment which ones are optional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nvm this is an example for postgres processor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Combining crates in a follow up pr
f2b3903
to
e213695
Compare
af1aa2d
to
8943b86
Compare
Create a Postgres events processor that uses predefined Postgres implementations for getting starting version and version tracking. The user-defined parts are:
2025-03-06-231718_create_events
Testing