-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
Private Data Comparison #2861
Private Data Comparison #2861
Conversation
a6a7f13
to
70e979b
Compare
@jcastrence You'll need to fix the staticcheck suggestions mentioned at https://dev.azure.com/Hyperledger/Fabric/_build/results?buildId=39933&view=logs&jobId=d3cbc547-3967-500e-4ad6-f35cc9436e08&j=d3cbc547-3967-500e-4ad6-f35cc9436e08&t=cb205cbf-7666-5009-0503-0a389840d9d5 |
70e979b
to
4976de0
Compare
a3861cd
to
a0cc6cb
Compare
internal/ledgerutil/compare.go
Outdated
s.highestRecord = (*s.records)[0] | ||
s.highestIndex = 0 | ||
} else { | ||
for i, r := range *s.records { |
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.
iterating through all the records every time is not an efficient way to do this. Better to keep the firstRecords sorted. But this could be improved in a follow-on PR since this is not the main intent of this PR.
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.
Agreed. The follow-on PR can implement a min-max heap to reduce runtime.
a0cc6cb
to
e255f33
Compare
Resolves hyperledger#2818 Signed-off-by: Julian Castrence <juliancastrence@ibm.com>
e255f33
to
bd3971b
Compare
Resolves #2818
Signed-off-by: Julian Castrence juliancastrence@ibm.com
Type of change
Description
Related issues