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

base reserve requirement for data fields is not sufficient for preventing the bloat #112

Open
s-a-y opened this issue May 8, 2018 · 2 comments
Labels
CAP Represents an issue that requires a CAP. help wanted Open especially for those who want to write a CAP/SEP! needs draft This an issue that has no corresponding draft, and as such has not entered the CAP/SEP process.

Comments

@s-a-y
Copy link
Contributor

s-a-y commented May 8, 2018

In this package https://www.npmjs.com/package/stellar-sign
I'm storing data into the blockchain:
TX:

  • manageData(key, value)
  • manageData(key, null)

I unset the data field right away, so I don't need to keep base reserve for each field I'm saving.
I can find transaction in transaction history for the account, so I don't need to have those fields set forever.

I think it is too cheap to bloat the ledger at the moment,
let's say I use 50 operations to store data, and another 50 to release base reserve.
50 * 128 bytes = 6400 bytes per transaction costs me 0.001 XLM (I'm not including tx itself, just pure data)
adding 6mb to the chain costs me 1 XLM
adding 6GB costs me 1000 XLM ($400 at current rate)

I think fees need to be adjusted for manageData operation specifically.

@jedmccaleb jedmccaleb added the CAP Represents an issue that requires a CAP. label Aug 3, 2018
@theaeolianmachine
Copy link
Contributor

@jedmccaleb @MonsieurNicolas — outside of pushing adoption of higher fees, is there anything else that we can do here? Definitely agree with the sentiment on it being very easy to bloat the ledger (moving towards IPFS & removing data entries might be the better route).

@theaeolianmachine
Copy link
Contributor

Mostly I'm looking for what steps should be taken next on this issue in your respective opinions.

@theaeolianmachine theaeolianmachine added help wanted Open especially for those who want to write a CAP/SEP! needs draft This an issue that has no corresponding draft, and as such has not entered the CAP/SEP process. labels Mar 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CAP Represents an issue that requires a CAP. help wanted Open especially for those who want to write a CAP/SEP! needs draft This an issue that has no corresponding draft, and as such has not entered the CAP/SEP process.
Projects
None yet
Development

No branches or pull requests

4 participants
@jedmccaleb @theaeolianmachine @s-a-y and others