Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Expensive teleports #2392

Closed
gilescope opened this issue Mar 28, 2023 · 0 comments · Fixed by paritytech/polkadot#6980
Closed

Expensive teleports #2392

gilescope opened this issue Mar 28, 2023 · 0 comments · Fixed by paritytech/polkadot#6980

Comments

@gilescope
Copy link
Contributor

#2102 has landed now that XCM v3 has arrived.

However the integration tests show that a limited_teleport_assets can be expensive (38bn weight). This is due to Wild(All) being used here: https://github.com/paritytech/polkadot/blob/bebd1a788ecccfa1a6adc61004e4f700988f40d4/xcm/pallet-xcm/src/lib.rs#L1207 - this means we multiply the weight by max assets (which is currently 100).

Could we change the code there so that it can sometimes issue Wild(AllOf) potentially as that has drastically lower fees?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant