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

account_referrer_index is empty #452

Closed
oxarbitrage opened this issue Nov 1, 2017 · 4 comments
Closed

account_referrer_index is empty #452

oxarbitrage opened this issue Nov 1, 2017 · 4 comments
Assignees
Labels
1b User Story The User Story details a requirement. It may reference a parent Epic. It may reference child Task(s) 2b Gathering Requirements Status indicating currently refining User Stories and defining Requirements 3c Enhancement Classification indicating a change to the functionality of the existing imlementation 6 API Impact flag identifying the application programing interface (API) 6 Performance Impacts flag identifying system/user efficiency, performance, etc. 6 Plugin Impact flag identifying at least one plugin code cleanup

Comments

@oxarbitrage
Copy link
Member

i was trying to make an api call for a way to get account referrers, something better than going every account client side checking the referrer field, i found there is an index for this purposes: account_referrer_index:

https://github.com/bitshares/bitshares-core/blob/master/libraries/chain/include/graphene/chain/account_object.hpp#L303

However this index it is not being populated: https://github.com/bitshares/bitshares-core/blob/master/libraries/chain/account_object.cpp#L257-L268

Unsure if this is just incomplete or empty by something else. our options will be to populate or to send account data to ES by plugin and index by anything needed.

@abitmore
Copy link
Member

abitmore commented Nov 8, 2017

I guess it's incomplete. IMHO account_referrer_index and account_member_index "should" be moved into a plugin, since they're not required by consensus.

//Update: according to this comment, it's not that easy to move them to a plugin.

@pmconrad
Copy link
Contributor

pmconrad commented Nov 8, 2017

The issue with undo_db is easily fixed, IMO. If any callbacks have been registered before the start of replay, enable undo_db, otherwise don't. Maybe log a warning about the performance hit when enabling.

@pmconrad
Copy link
Contributor

pmconrad commented Nov 8, 2018

Secondary indexes can easily be added in a plugin. No need to change undo_db behaviour during replay.

@pmconrad pmconrad added 1b User Story The User Story details a requirement. It may reference a parent Epic. It may reference child Task(s) 2b Gathering Requirements Status indicating currently refining User Stories and defining Requirements 3c Enhancement Classification indicating a change to the functionality of the existing imlementation 6 API Impact flag identifying the application programing interface (API) 6 Performance Impacts flag identifying system/user efficiency, performance, etc. 6 Plugin Impact flag identifying at least one plugin and removed question labels Nov 8, 2018
@pmconrad pmconrad added this to the Future Feature Release milestone Nov 8, 2018
@pmconrad pmconrad self-assigned this Aug 15, 2019
@abitmore
Copy link
Member

#1918 removed account_referrer_index and moved account_member_index to a plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1b User Story The User Story details a requirement. It may reference a parent Epic. It may reference child Task(s) 2b Gathering Requirements Status indicating currently refining User Stories and defining Requirements 3c Enhancement Classification indicating a change to the functionality of the existing imlementation 6 API Impact flag identifying the application programing interface (API) 6 Performance Impacts flag identifying system/user efficiency, performance, etc. 6 Plugin Impact flag identifying at least one plugin code cleanup
Projects
None yet
Development

No branches or pull requests

4 participants