-
Notifications
You must be signed in to change notification settings - Fork 66
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
Add checksum to speed up easyjson #91
Conversation
So now that you're not cleaning the gen-code dir, how are you going to remove excess files? |
I dont know how we are going to remove excess files. travis will fail if you dont remove them by hand. We cant clean the directory if we want to re-use checksum and have faster builds :( |
Well there is a process that checks the files, can you keep a list of the files that were checked or generated from scratch and remove anything not in that list? |
@Matt-Esch thats only for the |
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.
LGTM
oldEasyJSONBytes, err := ioutil.ReadFile(easyJSONFile) | ||
if err == nil { | ||
sliceStart := len(prefixBytes) + len(checksumPrefix) | ||
sliceEnd := len(prefixBytes) + len(checksumPrefix) + 24 |
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.
nit: const checksumLen = 24
could make 24 less magical.
I'll add the ability to remove generated files in a separate PR. |
EasyJSON can make
make generate
take upto 60 seconds.With the new checksum caching / short circuiting the
make generate
step now takes 10 seconds.r: @uber/zanzibar-team