-
Notifications
You must be signed in to change notification settings - Fork 262
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
Refactor processing of Internal Txs #2220
Conversation
with only_one_running_task(self, lock_name_suffix=safe_address): | ||
logger.info("[%s] Start processing decoded internal txs", safe_address) | ||
index_service: IndexService = IndexServiceProvider() | ||
number_processed = index_service.process_address(safe_address) |
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.
number_processed = index_service.process_address(safe_address) | |
number_processed = index_service.process_decoded_txs(safe_address) |
process_address sound for me too generic and confusing.
self.tx_processor.clear_cache(safe_address) | ||
|
||
# Use chunks for memory issues | ||
number_processed = 0 |
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.
number_processed = 0 | |
total_processed_txs = 0 |
For me number processed could indicate an ID for example, not the total amount of processed transactions.
- Task was refactored into a function in `IndexService` - Automatic reindexing of Safes was disabled. Reindexing issues should be fixed by the scheduled task `reindex_mastercopies_last_hours_task`. Also some Safes were corrupted due to users trying to trick the indexer or Safes converted from L1 -> L2, and reindexing was triggered forever
eba8b2f
to
35c5401
Compare
I've just applied your suggestions! |
35c5401
to
d8b78a4
Compare
IndexService
reindex_mastercopies_last_hours_task
. Also some Safes were corrupted due to users trying to trick the indexer or Safes converted from L1 -> L2, and reindexing was triggered forever