Skip to content
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

Cherry-pick #7859 to 6.4: Update go-txfile and dependencies #7911

Merged
merged 1 commit into from
Aug 8, 2018

Conversation

urso
Copy link

@urso urso commented Aug 8, 2018

Cherry-pick of PR #7859 to 6.4 branch. Original message:

Resolves #7849, #7867, #7720

This PR updates go-txfile to v0.0.3 and golang.org/x/sys packages (go-txfile requires more recent version).

The update fixes a number of Windows related bugs in spooling to disk and replaces a call to fsync with fcntl+F_FULLFSYNC on darwin.

The error messages generated by go-txfile will be more detailed. We will further improve error handling and error reporting in the spool in follow up PRs.

The changes to Changelog.asciidoc for all user visible fixes and improvements.

* Update go-txfile

* Update golang.org/x/sys

go-txfile requires a newer version of golang.org/x/sys

* Update spool to take API changes into account

* Ensure spool opening root cause is included when printing the error message

* cleanup vendor after govendor fails

* Update notice file

* Update changelog

* Fix build

* Update go-txfile to 0.0.3

(cherry picked from commit 10f41be)
@urso urso force-pushed the backport_7859_6.4 branch from e212041 to 9755be9 Compare August 8, 2018 13:54
@kvch kvch merged commit 2a5b90c into elastic:6.4 Aug 8, 2018
@urso urso deleted the backport_7859_6.4 branch February 19, 2019 18:36
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
* Update go-txfile

* Update golang.org/x/sys

go-txfile requires a newer version of golang.org/x/sys

* Update spool to take API changes into account

* Ensure spool opening root cause is included when printing the error message

* cleanup vendor after govendor fails

* Update notice file

* Update changelog

* Fix build

* Update go-txfile to 0.0.3

(cherry picked from commit b6e44a6)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants