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.

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

Cosmos SDK v46 #863

Closed
wants to merge 159 commits into from
Closed

Conversation

faddat
Copy link
Contributor

@faddat faddat commented May 17, 2022

Hi, so here's another CW for 46.

I am mainly interested in seeing how much better or worse we can get CW-enabled chains to run in high traffic scenarios.

I think this contributed to the fall of terra.

After some debugging and refinement, hope to get this out on Juno and Osmosis.

@faddat
Copy link
Contributor Author

faddat commented Jul 5, 2022

46 isn't released, but we'll probably start to use this on testnets, so I'm marking this r4r.

Thanks to everyone buildin' cw!

<3

@faddat faddat marked this pull request as ready for review July 5, 2022 16:30
@faddat faddat requested a review from alpe as a code owner July 5, 2022 16:30
@@ -47,13 +48,13 @@ func NewAnteHandler(options HandlerOptions) (sdk.AnteHandler, error) {
ante.NewSetUpContextDecorator(), // outermost AnteDecorator. SetUpContext must be called first
wasmkeeper.NewLimitSimulationGasDecorator(options.WasmConfig.SimulationGasLimit), // after setup context to enforce limits early
wasmkeeper.NewCountTXDecorator(options.TXCounterStoreKey),
ante.NewRejectExtensionOptionsDecorator(),
ante.NewMempoolFeeDecorator(),
// ante.NewRejectExtensionOptionsDecorator(),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I only think I did this right, pls let me know if it is wrong :)

@faddat
Copy link
Contributor Author

faddat commented Jul 5, 2022

Note: cosmos/cosmos-sdk#12458

@faddat
Copy link
Contributor Author

faddat commented Jul 6, 2022

<3 @vuong177

@faddat
Copy link
Contributor Author

faddat commented Jul 7, 2022

❤️ 🚀 @vuong177

@faddat faddat closed this Jul 23, 2022
@RafilxTenfen
Copy link

@faddat why did you closed?

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

Successfully merging this pull request may close these issues.

6 participants