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

[ADAM-1136] Transform runs successfully with kryo registration required #1138

Merged
merged 2 commits into from Aug 30, 2016

Conversation

Projects
None yet
3 participants
@fnothaft
Member

fnothaft commented Aug 28, 2016

Resolves #1136. Rewrote IndelRealignmentTarget/ZippedTargetSet serializers to resolve potential StackOverflowException.

Also, fixed one FileSystem.get that I'd missed in 4140095.

fnothaft added some commits Aug 28, 2016

[ADAM-1136] Transform runs successfully with kryo registration required
Resolves #1136. Rewrote IndelRealignmentTarget/ZippedTargetSet serializers to
resolve potential StackOverflowException.
@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Aug 29, 2016

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1432/
Test PASSed.

AmplabJenkins commented Aug 29, 2016

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1432/
Test PASSed.

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Aug 30, 2016

Member

LGTM

Member

heuermh commented Aug 30, 2016

LGTM

@heuermh heuermh merged commit 10ec74e into bigdatagenomics:master Aug 30, 2016

1 check passed

default Merged build finished.
Details
@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Aug 30, 2016

Member

Merged as commit 10ec74e.

Thank you, @fnothaft!

Member

heuermh commented Aug 30, 2016

Merged as commit 10ec74e.

Thank you, @fnothaft!

@fnothaft fnothaft deleted the fnothaft:issues/1136-transform-require-registration branch Sep 19, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment