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

Shards updates for v1.8.0 #1249

Merged
merged 2 commits into from
Nov 23, 2021
Merged

Shards updates for v1.8.0 #1249

merged 2 commits into from
Nov 23, 2021

Conversation

mDuo13
Copy link
Collaborator

@mDuo13 mDuo13 commented Nov 22, 2021

Documents XRPLF/rippled#3875, XRPLF/rippled#3595, and XRPLF/rippled#3726

  • Modifications to the crawl_shards command
  • New node_to_shard command
  • Updates concept page to mention deterministic shards
  • Bonus: Rewords some context about shards on the concept page and removes some outdated technical details

@mDuo13 mDuo13 added new topic Describes a detailed document to be added or expanded in detail content updates Updating, cleaning up, or rewriting existing content for better usability or new contexts labels Nov 22, 2021
@github-actions
Copy link

Link check report. 339245 links checked.
Success! No broken links found.

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/shards_180/

Style Report

Copy link
Contributor

@scottschurr scottschurr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not the expert on this, but it all looks right to me. I did have one suggestion which you can take or leave.

@undertome, could you review this as well?

| `complete_shards` | String | The range of history shards this peer has available. This may be disjointed. For example, `1-2,5,7-9` indicates that shards 1, 2, 5, 7, 8, and 9 are available. |
| `ip` | String | _(May be omitted)_ The IP address of the peer this object describes. This may be an IPv4 or IPv6 address. Omitted if this is a [private peer](peer-protocol.html#private-peers). |
| `complete_shards` | String | The range of complete history shards this peer has available. This may be disjointed. For example, `1-2,5,7-9` indicates that shards 1, 2, 5, 7, 8, and 9 are available. |
| `incomplete_shards` | String | _(May be omitted)_ A comma-separated list of history shards this peer has partially downloaded, and percent completion for each. For example, `1:50,2:25` indicates that shard 1 is 50% downloaded and shard 2 is 25% downloaded. [New in: rippled 1.8.0][] |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup, as far as I can tell this is correct.

- Data Retention
---
# node_to_shard
[[Source]](https://github.com/undertome/rippled/blob/develop/src/ripple/rpc/handlers/NodeToShard.cpp "Source")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This source address should probably be moved to the main rippled repository? Possibly this: https://github.com/ripple/rippled/blob/develop/src/ripple/rpc/handlers/NodeToShard.cpp

@github-actions
Copy link

Link check report. 339245 links checked.
Success! No broken links found.

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/shards_180/

Style Report

Copy link

@undertome undertome left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mDuo13 mDuo13 merged commit 8aafce4 into XRPLF:rippled_180 Nov 23, 2021
@mDuo13 mDuo13 deleted the shards_180 branch November 23, 2021 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content updates Updating, cleaning up, or rewriting existing content for better usability or new contexts new topic Describes a detailed document to be added or expanded in detail
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants