near / mpc

37 stars 11 forks source link

gas test #834

Closed volovyks closed 1 week ago

ailisp commented 2 weeks ago

Nice finding! I would also test using a maximum sized request

volovyks commented 2 weeks ago

@ailisp what do you mean by maximum sized request? Aren't they all the same size?

ailisp commented 2 weeks ago

@volovyks i mean path. there was some clear_state_on_finish invoked by sign_helper spending around 6T gas while it usually spend 3T gas. Given the clear_state_on_finish is very simple, it maybe related to size of request

volovyks commented 1 week ago

@ailisp good point, I've checked, and path size does affect gas, but it is not significant for now.