Remove write lock option from the platformvm API #2157
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
StephenButtolph
added
consensus
gyuho
approved these changes
Oct 10, 2023
abi87
reviewed
Oct 11, 2023
abi87
approved these changes
Oct 11, 2023
joshua-kim
added a commit
that referenced
this pull request
Oct 17, 2023
commit 188f2b2 Author: David Boehm <91908103+dboehm-avalabs@users.noreply.github.com> Date: Mon Oct 16 13:10:00 2023 -0400 MerkleDB Reduce buffer creation/memcopy on path construction (#2124) Co-authored-by: Dan Laine <daniel.laine@avalabs.org> commit 9d44ec2 Author: Alberto Benegiamo <alberto.benegiamo@gmail.com> Date: Mon Oct 16 08:53:59 2023 -0700 Validator Diffs: docs and UTs cleanup (#2037) Co-authored-by: Stephen Buttolph <stephen@avalabs.org> commit 50f131e Author: Patrick O'Grady <prohb125@gmail.com> Date: Thu Oct 12 23:04:02 2023 -0700 [x/merkledb] `Prefetcher` interface (#2167) commit 007f98d Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Thu Oct 12 22:33:23 2023 -0700 Bump google.golang.org/grpc from 1.55.0 to 1.58.3 (#2159) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Stephen Buttolph <stephen@avalabs.org> commit 8d2c4f2 Author: Stephen Buttolph <stephen@avalabs.org> Date: Thu Oct 12 19:44:04 2023 -0400 Use set.Of rather than set.Add (#2164) commit 9da2e62 Author: Stephen Buttolph <stephen@avalabs.org> Date: Thu Oct 12 17:41:27 2023 -0400 Remove context lock from API VM interface (#2165) commit 9dbf82a Author: Stephen Buttolph <stephen@avalabs.org> Date: Wed Oct 11 19:48:54 2023 -0400 Remove write lock option from the avm rpc API (#2156) commit 2eb6e84 Author: Stephen Buttolph <stephen@avalabs.org> Date: Wed Oct 11 19:39:44 2023 -0400 Remove write lock option from the platformvm API (#2157) commit 9725095 Author: Dhruba Basu <7675102+dhrubabasu@users.noreply.github.com> Date: Wed Oct 11 14:12:29 2023 -0700 Remove aliasing of `math` standard lib (#2163) commit 18fbdef Author: Stephen Buttolph <stephen@avalabs.org> Date: Wed Oct 11 16:57:11 2023 -0400 Remove lock options from the admin API (#2150) commit aae7260 Author: Stephen Buttolph <stephen@avalabs.org> Date: Wed Oct 11 16:56:13 2023 -0400 Remove lock options from the IPCs api (#2151) commit 8247f74 Author: Stephen Buttolph <stephen@avalabs.org> Date: Wed Oct 11 16:56:05 2023 -0400 Remove write lock option from the xsvm API (#2152) commit 1bc63d4 Author: Dhruba Basu <7675102+dhrubabasu@users.noreply.github.com> Date: Wed Oct 11 16:34:12 2023 -0400 Rename `removeSubnetValidatorValidation` to `verifyRemoveSubnetValidatorTx` (#2162) commit 99fc926 Author: Stephen Buttolph <stephen@avalabs.org> Date: Wed Oct 11 12:39:01 2023 -0400 Fix json marshalling of Sets (#2161) commit 0f95f13 Author: Stephen Buttolph <stephen@avalabs.org> Date: Wed Oct 11 11:37:57 2023 -0400 Remove write lock option from the avm wallet API (#2155) commit e6dab5d Author: Stephen Buttolph <stephen@avalabs.org> Date: Wed Oct 11 11:30:41 2023 -0400 Remove write lock option from the avm static API (#2154) commit 7f61fee Author: Stephen Buttolph <stephen@avalabs.org> Date: Tue Oct 10 23:17:17 2023 -0400 Remove lock options from the info api (#2149) commit c50ea11 Author: Stephen Buttolph <stephen@avalabs.org> Date: Tue Oct 10 23:13:19 2023 -0400 Marshal blocks and transactions inside API calls (#2153) commit 0ac1937 Author: kyoshisuki <143475866+kyoshisuki@users.noreply.github.com> Date: Tue Oct 10 20:12:25 2023 -0700 Fix typo in block formation logic documentation (#2158) commit 145dfb0 Author: Stephen Buttolph <stephen@avalabs.org> Date: Tue Oct 10 22:18:27 2023 -0400 Update versions for v1.10.12 (#2139) commit 6d53e51 Author: Stephen Buttolph <stephen@avalabs.org> Date: Fri Oct 6 18:08:15 2023 -0400 Split Alpha into AlphaPreference and AlphaConfidence (#2125) commit 1fc8973 Author: Stephen Buttolph <stephen@avalabs.org> Date: Thu Oct 5 17:50:20 2023 -0400 Add additional payload.Hash examples (#2145) Signed-off-by: Stephen Buttolph <stephen@avalabs.org> Co-authored-by: Dhruba Basu <7675102+dhrubabasu@users.noreply.github.com> Signed-off-by: Joshua Kim <20001595+joshua-kim@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why this should be merged
Factored out of #2148
How this works
Removes the usage of
common.WriteLockand explicitly grabs the lock when handling allplatformvmAPIsHow this was tested
CI