[api] archive-mode for eth_getBalance & eth_call #4529
Conversation
dustinxie
requested review from
CoderZhi,
Liuhaai,
envestcc and
a team
as code owners
| type intrinsicGasCalculator interface { | ||
| IntrinsicGas() (uint64, error) | ||
| } | ||
|
|
||
| var ( | ||
| // ErrNotFound indicates the record isn't found | ||
| ErrNotFound = errors.New("not found") | ||
| ErrNotFound = errors.New("not found") | ||
| ErrArchiveNotSupported = errors.New("archive-mode not supported") |
There was a problem hiding this comment.
removed L261 comment, the name clears explains what the error is
| } | ||
|
|
||
| func (core *coreServiceReaderWithHeight) Account(addr address.Address) (*iotextypes.AccountMeta, *iotextypes.BlockIdentifier, error) { | ||
| if !core.cs.archiveSupported { |
There was a problem hiding this comment.
bd7ca1f#diff-699e59a265a0599d1a49698bbf378f57a7aeb60957441a257249102ac8785a8dR50 will return error if archive is not supported
|
|
||
| type ( | ||
| // CoreServiceReaderWithHeight is an interface for state reader at certain height | ||
| CoreServiceReaderWithHeight interface { |
There was a problem hiding this comment.
could you explain why we need to create such an interface?
There was a problem hiding this comment.
this interface is created to represent APIs needed for archive-mode query (there will be a total 4-6 of them, 2 enabled in this PR, and more to enable in upcoming PR), so we can keep using existing API without having to add the height parameter to it, that is, we can avoid changes like below
Account(address.Address, height uint64) (*iotextypes.AccountMeta, *iotextypes.BlockIdentifier, error)
| if err != nil { | ||
| return nil, nil, status.Error(codes.Internal, err.Error()) | ||
| } | ||
| ws, err = core.sf.WorkingSet(ctx) |
There was a problem hiding this comment.
these lines would be duplicated multiple times if use ws instead of boolean as input
|
| var ( | ||
| accountMeta *iotextypes.AccountMeta | ||
| height, archive = blockNumberToHeight(bn) | ||
| ) | ||
| if !archive { | ||
| accountMeta, _, err = svr.coreService.Account(ioAddr) | ||
| } else { | ||
| accountMeta, _, err = svr.coreService.WithHeight(height).Account(ioAddr) | ||
| } |
There was a problem hiding this comment.
such that we can always return a Reader from svr.coreService by calling svr.coreService.Reader(bn).Account(ioAddr)
Description
as title. Based on #4526
Fixes #(issue)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist: