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.

Sign up for GitHub

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

Jump to bottom

Failed to get XMP balance. #29

Open
cryptcoin-junkey opened this issue Sep 13, 2021 · 4 comments · May be fixed by #30
Open

Failed to get XMP balance. #29

cryptcoin-junkey opened this issue Sep 13, 2021 · 4 comments · May be fixed by #30

Comments

@cryptcoin-junkey
Copy link
Member

https://twitter.com/jung2neko/status/1437410136000110596

■モナパレットでトークン発行を行ったら、いままで表示されていたXMPの残高が0になった。
トリガとなったトランザクション　→　mpchain.info/tx/229710
@cryptcoin-junkey
Copy link
Member Author

https://twitter.com/jung2neko/status/1437410314895519749

■MpchainやMpurseではXMP残高があるように表示される。
mpchain.info/address/MN9V3VjdU6iM77DzoGcKXkpezgRDVqjung
■すなぎもさんに相談したところ、モナパレットではget_normalized_balancesを使用しており、アドレス内に保持しているトー> クンの種類が多くなりすぎると、
全件取得できなくなるとのこと。

image

@cryptcoin-junkey
Copy link
Member Author

https://twitter.com/jung2neko/status/1437410474274951168

■トークンの種類を 896 → 471　まで減らしたが、XMPは0のまま。
■Mpurseを使って、該当アドレスのXMPの出し入れをしたが、モナパレットの残高は0XMPのまま。

https://twitter.com/jung2neko/status/1437410621104947204

■オーナートークンが上限超過した可能性を考慮し、該当アドレスがオーナーとなっているトークンのオーナー権限を他アドレスに移譲するも、0XMPのまま。

@anipopina
Copy link

counterblock/counterblock/lib/modules/assets.py

Lines 147 to 149 in 0b94e65

result = util.call_jsonrpc_api(
"get_balances",
{'filters': filters, 'filterop': 'or'}, abort_on_error=True)['result']

get_normalized_balancesの処理において、ページング指定なしでget_balancesしています。

balancesテーブルにはquantityが0になったレコードも残り続けるため、ひとたび1ページ目からあふれてしまったレコードはその後アドレスの中身を整理しても1ページ目に戻ってきません。
利用頻度の多いアドレスにおいてXMPを含む一部アセットの情報がget_normalized_balancesで恒久的に取得できなくなったのは、これが理由と思われます。

@cryptcoin-junkey
Copy link
Member Author

@anipopina ちょうどいま、その辺りを眺めていたところでした。
現象はご指摘の通りと思いますが、そもそも counterparty server に問い合わせているのが設計として不適切(高負荷の元)という感もあり、dispenser 周りも含め見直しが必要かと思っています。

@cryptcoin-junkey cryptcoin-junkey linked a pull request Sep 16, 2021 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use pagenation on getting assets in get_normalized_balances. monaparty/counterblock
2 participants
@cryptcoin-junkey @anipopina