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

DP: Document v-ops presence/absence when streaming #257

Closed
3 tasks done
relativityboy opened this issue Jul 16, 2018 · 4 comments
Closed
3 tasks done

DP: Document v-ops presence/absence when streaming #257

relativityboy opened this issue Jul 16, 2018 · 4 comments
Assignees
Labels
5 - SW Steemworks Points

Comments

@relativityboy
Copy link
Contributor

relativityboy commented Jul 16, 2018

@inertia186 commented:

As an application developer, I want to be able to stream virtual ops from head_block_num instead of last_irreversible_block_num, because I want a live view of what's happening.

head_block_num does not have v-ops.

AC

  • document in the DP that this is not the case
  • document on streaming tutorial that "currently" while streaming from head is possible, it's an exceptionally bad idea because you lose v-ops (phrase appropriately). Comment that this may change in future (allude to bug, or link to bug)
  • admonish against use of head_block_num in general
@roboza roboza added the 5 - SW Steemworks Points label Jul 16, 2018
@inertia186
Copy link
Contributor

This appears to be an artifact of jussi.

Streaming vops on head is a problem that is mostly mitigated by adding a 3 second wait when its encountered. On occasion, it takes a bit longer than 3 seconds.

However, it's not just a problem on the head block. I've also encountered it on irreversible blocks, very rarely.

For my client (steem-ruby), I've added a check when streaming vops, that if it encounters a response of empty vops, it waits 3 seconds and retries. This can cause the stream to fall behind. But it can always catch up by requesting blocks with json-rpc-batch, so it's no big deal.

@inertia186
Copy link
Contributor

Seems related: steemit/steem#2690

@roboza roboza self-assigned this Aug 17, 2018
relativityboy added a commit that referenced this issue Aug 28, 2018
relativityboy pushed a commit that referenced this issue Aug 28, 2018
* virtual ops streaming recipe

* Fixing formatting

* changed wording

* [#257] minor lang update
@inertia186
Copy link
Contributor

Additional strategy in HF20: steemit/steem#2099 (comment)

inertia186 pushed a commit to inertia186/hivedocs that referenced this issue Dec 18, 2018
* virtual ops streaming recipe

* Fixing formatting

* changed wording

* [steemit#257] minor lang update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - SW Steemworks Points
Projects
None yet
Development

No branches or pull requests

4 participants