-
Notifications
You must be signed in to change notification settings - Fork 38
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
[PIPE-1131] Zip shared object mapping files #127
Conversation
|
||
return true | ||
zipStream.close() | ||
stdout.close() |
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.
it'd be good to close this in a finally block (there's an open PR which cleans this up generally across the plugin)
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.
changed
} catch (IOException e) { | ||
project.logger.error("failed to write symbols for " + arch + ": " + e.getMessage()) | ||
return false | ||
byte[] buffer = new byte[1024]; |
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.
BufferedOutputStream defaults to 8192
, we could probably increase this to speed up IO
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.
changed
@@ -122,7 +123,7 @@ class BugsnagUploadNdkTask extends BugsnagMultiPartUploadTask { | |||
outputDir.mkdir() | |||
} | |||
|
|||
File outputFile = new File(outputDir, arch + ".txt") | |||
File outputFile = new File(outputDir, arch + ".gzip") |
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.
[Nitpick] .gz is the typical file extension for gzipped files
Roll forward in release commit
Goal
Zip shared object mapping files before sending, to reduce transfer time. Also don't process the file as zipping is very effective
This coresponding PR is required to process the zipped files
https://github.com/bugsnag/bugsnag-event-worker/pull/1043
Design
See ROAD-570 design for reasons for the approach