steveloughran opened a new pull request, #3204:
URL: https://github.com/apache/parquet-java/pull/3204
Downgrade flush in HadoopPositionOutputStream.close() from Syncable.hflush()
to a basic flush() operation.
### Rationale for this change
calling Syncable.sync() is time-consuming overkill just before a close()
operation, and trouble on other stores (s3a will log at warn, when not
configured to fail fast)
### What changes are included in this PR?
Switches to classic OutputStream.flush()
### Are these changes tested?
Expecting test runner to do this.
### Are there any user-facing changes?
No
<!-- Please uncomment the line below and replace ${GITHUB_ISSUE_ID} with the
actual Github issue id. -->
Closes #3203
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
