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
[ZOOKEEPER-3387] Use BufferedWriter instead of FileWriter #941
base: master
Are you sure you want to change the base?
Conversation
Though the build checks have failed due to other reason, the BufferedWriter -> FileWriter changes still remain valid and applicable to the code base |
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.
This patch affects only build time.
Do you see any real improvement in build time?
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.
This patch affects only build time.
Do you see any real improvement in build time?
retest this please |
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.
As outlined by @eolivelli this is a build-time only improvement. I appreciate your contribution @bd2019us , but Jute is not maintained anymore and we want to replace it in the long run.
How much time do you gain in build time with this change?
@bd2019us This patch doesn't compile:
|
ZOOKEEPER-3387 When FileWriter is used intensively within a loop, BufferedWriter can be used to improve performance by reducing IO operations.