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

add check-tree-leaf to tree-status #19

Merged
merged 3 commits into from
May 13, 2023
Merged

add check-tree-leaf to tree-status #19

merged 3 commits into from
May 13, 2023

Conversation

fanatid
Copy link
Collaborator

@fanatid fanatid commented May 10, 2023

Compare maximum seq number per each leaf with assets table in database

example:

     Running `target/debug/tree-status --rpc {RPC} check-tree-leafs -p {POSTGRES} -t 8wKvdzBu2kEG5T3maJBX8m2gLs4XFavXzCKiZcGVeS8T`
checking tree leafs 8wKvdzBu2kEG5T3maJBX8m2gLs4XFavXzCKiZcGVeS8T
b6f35ce4a304f88a74966628420d8a965a19c7e02af9f350dd8fe16c91d4bc70 247: not found in db, seq 248
0000000000000000000000000000000000000000000000000000000000000000 0: not found in db, seq 0
d6f9e2cea59a9a3a1927a734bc28be0bfacc725fe36faac712aeb3a0ab2db068 525: not found in db, seq 526

@fanatid fanatid requested a review from linuskendall May 10, 2023 14:24
@fanatid fanatid marked this pull request as ready for review May 13, 2023 02:45
@fanatid fanatid changed the title [WIP] add check-tree-leaf to tree-status add check-tree-leaf to tree-status May 13, 2023
@linuskendall linuskendall merged commit 419cc76 into triton May 13, 2023
@fanatid fanatid deleted the check-tree2 branch May 13, 2023 13:15
linuskendall added a commit that referenced this pull request Jun 22, 2023
* das: error if the page is 0 (#18)

* DAS-API: Reject requests if the fields are unknown (#19)

---------

Co-authored-by: Nikhil Acharya <[email protected]>
linuskendall added a commit that referenced this pull request Jun 22, 2023
* das: error if the page is 0 (#18)

* DAS-API: Reject requests if the fields are unknown (#19)

---------

Co-authored-by: Nikhil Acharya <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants