Add runtime support for address table lookups #22223
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jstarry
force-pushed
the
address-table-lookups
branch
from
5f1096a to
d139400
Compare
jstarry
force-pushed
the
address-table-lookups
branch
from
d139400 to
ff20b52
Compare
jstarry
force-pushed
the
address-table-lookups
branch
from
ff20b52 to
e3a3e31
Compare
Codecov Report
@@ Coverage Diff @@
## master #22223 +/- ##
=========================================
- Coverage 81.1% 81.0% -0.1%
=========================================
Files 551 551
Lines 149828 150085 +257
=========================================
+ Hits 121522 121695 +173
- Misses 28306 28390 +84 |
jstarry
force-pushed
the
address-table-lookups
branch
from
e3a3e31 to
8b3f258
Compare
jstarry
force-pushed
the
address-table-lookups
branch
from
8b3f258 to
6eb62bf
Compare
jstarry
force-pushed
the
address-table-lookups
branch
2 times, most recently
from
67104ef to
36e5929
Compare
jstarry
force-pushed
the
address-table-lookups
branch
2 times, most recently
from
a09d84c to
4c6f9f2
Compare
joncinque
previously approved these changes
Jan 5, 2022
jstarry
force-pushed
the
address-table-lookups
branch
from
91cf5bb to
7ac2e3c
Compare
mergify
bot
dismissed
joncinque’s stale review
Pull request has been modified.
jstarry
force-pushed
the
address-table-lookups
branch
from
7ac2e3c to
dce2c73
Compare
joncinque
previously approved these changes
Jan 6, 2022
mergify
bot
dismissed
joncinque’s stale review
Pull request has been modified.
mergify bot
pushed a commit
that referenced
this pull request
Jan 7, 2022
* Add support for address table lookups in runtime * feedback * feedback (cherry picked from commit 52d12cc) # Conflicts: # accountsdb-plugin-postgres/scripts/create_schema.sql # runtime/src/accounts.rs # runtime/src/bank.rs # runtime/src/message_processor.rs
|
@jackcmay it'd still be nice to get your eyes on this if you can find the time, I merged to get a dependent change up for review |
|
I started working it and got multiple interrupts, will return |
jstarry
added a commit
to jstarry/solana
that referenced
this pull request
Jan 7, 2022
jstarry
added a commit
to jstarry/solana
that referenced
this pull request
Jan 7, 2022
jstarry
added a commit
to jstarry/solana
that referenced
this pull request
Jan 7, 2022
jstarry
added a commit
that referenced
this pull request
Jan 7, 2022
65 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary of Changes
This is an incremental change to supporting dynamic address loading for transactions that make use of address table lookups. This change stops short of allowing processed versioned transactions from being persisted to blockstore. That change will come in a follow up PR.
AddressLookupTablestate accounts