Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor: optimize performance of queryBtcTimeLockTxByBtcTx() #212

Merged
merged 3 commits into from
Aug 23, 2024

Conversation

ShookLyngs
Copy link
Collaborator

@ShookLyngs ShookLyngs commented Aug 22, 2024

Copy link

vercel bot commented Aug 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
btc-assets-api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 22, 2024 1:11pm

@ShookLyngs ShookLyngs changed the title refactor: optimize performance of RgbppCollector.queryBtcTimeLockTxByBtcTxId() refactor: optimize performance of queryBtcTimeLockTxByBtcTxId() Aug 22, 2024
Copy link
Collaborator

@ahonn ahonn left a comment

Choose a reason for hiding this comment

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

LGTM

src/services/rgbpp.ts Outdated Show resolved Hide resolved
@Flouse Flouse changed the title refactor: optimize performance of queryBtcTimeLockTxByBtcTxId() refactor: optimize performance of queryBtcTimeLockTxByBtcTx() Aug 22, 2024
@Dawn-githup
Copy link
Collaborator

Performance Bottleneck Optimization Verification (L1—L2)

Before Optimization:

  • 745 ms
  • 755 ms
  • 635 ms
  • 602 ms
  • 580 ms

After Optimization:

  • 335 ms
  • 332 ms
  • 349 ms
  • 340 ms
  • 337 ms

@ShookLyngs ShookLyngs merged commit 6ad5695 into develop Aug 23, 2024
2 of 4 checks passed
@ShookLyngs ShookLyngs deleted the ref/btc-time-lock-cells-searching branch August 23, 2024 12:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Optimize the search process for BTC_TIME_LOCK ckb transactions
4 participants