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
Rollback #1759 #1798
Rollback #1759 #1798
Conversation
a4bd157
to
ff1b543
Compare
@riemannulus Could you elaborate some context/background behind this patch in the pull request description? |
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.
ff1b543
to
eae593e
Compare
6fc0975
to
b7a1d20
Compare
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
Since #1759, we had a problem with preload. So I was reverted.
If you look at the following code, there is a part where you made a mistake while deleting the parameter.
https://github.com/planetarium/libplanet/pull/1792/files#diff-215bb45126357b0974dea14401f0213b497011890a129ae3a7df3e3468ea4911L524
If preload is always true, this
if
statement is meanless but I was left this code blocks.In this PR. I'm re-implement #1759 and fixing those bugs.