-
Notifications
You must be signed in to change notification settings - Fork 79
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
Migrate Parquet Events Processor to SDK #618
Conversation
rust/processor/src/db/postgres/models/events_models/parquet_events.rs
Outdated
Show resolved
Hide resolved
rust/processor/src/db/postgres/models/events_models/raw_events.rs
Outdated
Show resolved
Hide resolved
rust/processor/src/worker.rs
Outdated
// More tables | ||
const CURRENT_TABLE_ITEMS = 1 << 24; | ||
const BLOCK_METADATA_TRANSACTIONS = 1 << 25; | ||
|
||
// Events | ||
const EVENTS = 1 << 24; |
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: 24 is used by current table items. maybe this will likely happen again, should we assign specific range to each processor to prevent this conflict? like 20-30 -> default processor 30-40 -> fa. since it will be less than 10 tables
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.
oh. that's a good idea.
Purpose
RawEvent
model andEventConvertible
trait for shared parsing logic.RawEvent.from_raw_event
is used byevents
andparquet_events
.process_transactions_parquet
.Testing