test: use address lookup table in max refund test #620
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.
Increases max refund count by using Address Lookup Table (ALT). This is capped at 19 refunds as higher count causes out of memory panic when looping through refund transfers. To overcome this limit we would need to split individual refund claims in separate transaction. And if we split this, then we don't really need to use ALT.
Edit: limit is now increased to 21 by dropping expensive debug msg of input data. Now the bottleneck is calldata size, but that could be resolved by passing it from a buffer account.