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.

Sign up for GitHub

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

Jump to bottom

perf: eth: gas estimate set applyTsMessages false #10546

Merged
merged 7 commits into from
Mar 23, 2023
Merged

perf: eth: gas estimate set applyTsMessages false #10546

merged 7 commits into from
Mar 23, 2023

Commits on Mar 23, 2023

  1. have gas estimate call callInternal with applyTsMessages = false and … 

    …other calls with applyTsMessages=true for gas caclulation optimization
    @snissn
    snissn committed Mar 23, 2023
    Configuration menu
    Copy the full SHA
    b43bcc0 View commit details
    Browse the repository at this point in the history

  2. set applyTsMessages = true in CallWithGas call in shed

    @snissn
    snissn committed Mar 23, 2023
    Configuration menu
    Copy the full SHA
    f262462 View commit details
    Browse the repository at this point in the history

  3. update test with new callwithgas api optimization for eth call

    @snissn
    snissn committed Mar 23, 2023
    Configuration menu
    Copy the full SHA
    5762a30 View commit details
    Browse the repository at this point in the history

  4. Update chain/stmgr/call.go 

    Co-authored-by: Łukasz Magiera <magik6k@users.noreply.github.com>
    @jennijuju @magik6k
    jennijuju and magik6k authored Mar 23, 2023
    Configuration menu
    Copy the full SHA
    1f6b5b7 View commit details
    Browse the repository at this point in the history

  5. env flag LOTUS_SKIP_APPLY_TS_MESSAGE_CALL_WITH_GAS must be 1 in order… 

    … to have applyTsMessages change
    @snissn
    snissn committed Mar 23, 2023
    Configuration menu
    Copy the full SHA
    412c1e0 View commit details
    Browse the repository at this point in the history

  6. env flag LOTUS_SKIP_APPLY_TS_MESSAGE_CALL_WITH_GAS must be 1 in order… 

    … to have applyTsMessages change
    @snissn
    snissn committed Mar 23, 2023
    Configuration menu
    Copy the full SHA
    85b5ab4 View commit details
    Browse the repository at this point in the history

  7. make sure that even if we arent apply ts messages we grab ts messages… 

    … from the particular user who is requesting gas estimation
    Ubuntu committed Mar 23, 2023
    Configuration menu
    Copy the full SHA
    d770e18 View commit details
    Browse the repository at this point in the history