The AI Works community logo The Blockchain Works community logo The Functional Works community logo The Golang Works community logo The Java Works community logo The JavaScript Works community logo The Python Works community logo The Remote Works community logo The WorksHub company logo

We use cookies and other tracking technologies to improve your browsing experience on our site, analyze site traffic, and understand where our audience is coming from. To find out more, please read our privacy policy.

By choosing 'I Accept', you consent to our use of cookies and other tracking technologies.

We use cookies and other tracking technologies to improve your browsing experience on our site, analyze site traffic, and understand where our audience is coming from. To find out more, please read our privacy policy.

By choosing 'I Accept', you consent to our use of cookies and other tracking technologies. Less

We use cookies and other tracking technologies... More

Login or register
to publish this job!

Login or register
to save this job!

Login or register
to save interesting jobs!

Login or register
to get access to all your job applications!

Login or register to start contributing with an article!

Login or register
to see more jobs from this company!

Login or register
to boost this post!

Show some love to the author of this blog by giving their post some rocket fuel 🚀.

Login or register to search for your ideal job!

Login or register to start working on this issue!

Login or register
to save articles!

Login to see the application

Engineers who find a new job through Functional Works average a 15% increase in salary 🚀

You will be redirected back to this page right after signin

Allow configuring LRU cache size in app.toml

Issue closed
Pull requests: 1
Contributors: 0
Level: Intermediate
  • Go
Issue closed
Pull requests: 1
Contributors: 0
Level: Intermediate
  • Go

On GitHub

Basic cosmos-sdk app with web assembly smart contracts
More info >

Issue posted by: 
ethanfrey's avatar

Ethan Frey

Description

Summary

Node operators should be able to configure LRU cache size based on their memory availability

Problem Definition

Storing instances in the LRU will have no effect on the results (still deterministic), but should lower execution time at the cost of increased memory usage. We cannot pick universal parameters for this, so we should allow node operators to set it.

Proposal

There seems to be an app.toml that the node operator can configure some subjective values like halt-height and min-gas-fee or such. Let's add one more value:

[wasm]
lru_size = 10

The default should be 0, which means disabled. This value must be parsed at start and is passed into the wasm.NewKeeper function. We currently have hardcoded the value to 3: https://github.com/cosmwasm/wasmd/blob/master/x/wasm/internal/keeper/keeper.go#L44


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
  • API

Use Open Source to hire or get hired

On GitHub

Basic cosmos-sdk app with web assembly smart contracts
More info >

Issue posted by: 
ethanfrey's avatar

Ethan Frey

Use Open Source to hire or get hired

Allow configuring LRU cache size in app.toml
View on GitHub