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.

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

feat(runtime): address lookup table #598

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from
Draft

Conversation

Sobeston
Copy link
Contributor

@Sobeston Sobeston commented Mar 4, 2025

Implementation:

  • CreateLookupTable
    - need to overwrite some account data
    - need to do transfers
    - need to do allocation
  • FreezeLookupTable
    - need to overwrite some account data
  • ExtendLookupTable
    - need to overwrite some account data
    - need to do transfers
  • DeactivateLookupTable
    - need to overwrite some account data
  • CloseLookupTable

Testing:

  • CreateLookupTable
  • FreezeLookupTable
  • ExtendLookupTable
  • DeactivateLookupTable
  • CloseLookupTable

Copy link
Contributor Author

Sobeston commented Mar 4, 2025

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.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Sobeston Sobeston mentioned this pull request Mar 4, 2025
@Sobeston Sobeston changed the title WIP feat(account lookup table) Mar 4, 2025
@Sobeston Sobeston force-pushed the sobe/address-lookup-table branch 3 times, most recently from 90ad661 to 501a5a0 Compare March 4, 2025 23:04
@0xNineteen 0xNineteen changed the title feat(account lookup table) feat(runtime): account lookup table Mar 5, 2025
@0xNineteen 0xNineteen changed the title feat(runtime): account lookup table feat(runtime): address lookup table Mar 5, 2025
@yewman yewman force-pushed the harnew/native-cpi branch from 703f939 to 574dd7b Compare March 6, 2025 18:55
Base automatically changed from harnew/native-cpi to main March 6, 2025 21:11
@Sobeston Sobeston force-pushed the sobe/address-lookup-table branch from 501a5a0 to 4de9fad Compare March 7, 2025 01:13
Copy link

codecov bot commented Mar 7, 2025

Codecov Report

Attention: Patch coverage is 0.72993% with 272 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...c/runtime/program/address_lookup_table/execute.zig 0.00% 256 Missing ⚠️
src/runtime/program/system_program/lib.zig 0.00% 6 Missing ⚠️
src/runtime/sysvar/slot_hashes.zig 0.00% 6 Missing ⚠️
src/core/instruction.zig 0.00% 4 Missing ⚠️
Files with missing lines Coverage Δ
src/runtime/program/address_lookup_table/lib.zig 100.00% <100.00%> (ø)
src/core/instruction.zig 0.00% <0.00%> (ø)
src/runtime/program/system_program/lib.zig 0.00% <0.00%> (ø)
src/runtime/sysvar/slot_hashes.zig 0.00% <0.00%> (ø)
...c/runtime/program/address_lookup_table/execute.zig 0.00% <0.00%> (ø)

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🏗 In progress
Development

Successfully merging this pull request may close these issues.

1 participant