-
Notifications
You must be signed in to change notification settings - Fork 11.3k
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
[follower api] Augments the follower API with transaction info #1446
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1446 +/- ##
==========================================
+ Coverage 82.19% 82.31% +0.12%
==========================================
Files 104 103 -1
Lines 21145 21100 -45
==========================================
- Hits 17380 17369 -11
+ Misses 3765 3731 -34
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree this works, and we can use this to start implementing things like event subscription, but this is going to blow up the authority at any decent TPS. We should at least add this to the benchmarks.
Once we have a multi-get for getting transaction info from a vector of transactions we can adapt it to use that. |
eb7eaa2
to
e1b92b0
Compare
…ading the transactions information
e1b92b0
to
57a15a0
Compare
* refactor * clean up * refactor rebase * rebase * rebase * add how-to * format * lint * don't pop key, copy it * Add a safe client function that augments the follower API with downloading the transactions information (#1446) Co-authored-by: George Danezis <[email protected]> * Removing dev-addresses from Move.toml (#1536) There is no undefined address in the [addresses] section (e.g. AddressToBeFilledIn = "_") so having a [dev-addresses] section is not necessary. * [move][adapter] Static init rules (#1532) - Made init rules static - The existence of the function is optional - The function must be named 'init' - The function must be private - The function can have a single parameter, &mut TxContext - Alternatively, the function can have zero parameters * adds a counter example (#1539) * sui: reorganize binaries * wallet: log git revision on start * Refactor Build index by common user workflow (#1530) * Refactor Build index by common user workflow * Update index.md Fix paths missing Markdown file extension * Update index.md (#1545) Removing tutorial series until ready * Update index.md Remove version from API link Capitalize REST * comments and add unwind logic for process * fix: remove one unneeded instance of key_pair.copy() * fix: remove three unneeded instances of key_pair.copy() * fix: remove one unneeded instance of key_pair.copy() * refactor rebase * rebase * refactor * rebase * rebase * add how-to * lint * don't pop key, copy it * rebase * Update index.md Remove version from API link Capitalize REST * rebase Co-authored-by: Lu Zhang <[email protected]> Co-authored-by: George Danezis <[email protected]> Co-authored-by: George Danezis <[email protected]> Co-authored-by: jaredcosulich <[email protected]> Co-authored-by: Todd Nowacki <[email protected]> Co-authored-by: Damir S <[email protected]> Co-authored-by: Brandon Williams <[email protected]> Co-authored-by: Clay-Mysten <[email protected]> Co-authored-by: François Garillot <[email protected]>
We add a
safe_client
function that augments the follower API stream with a call to download eachtransaction info
for received digests.