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-1186] Mask null from fs.globStatus. #1187

Merged
merged 1 commit into from Sep 28, 2016

Conversation

Projects
None yet
3 participants
@fnothaft
Member

fnothaft commented Sep 28, 2016

Resolves #1186. Check for a null pointer in ADAMContext.getFiles. Make ADAMContext.getFsAndFiles package private so that we can unit test it. Added unit tests to check for proper exception with empty directories and bad globs.

[ADAM-1186] Mask null from fs.globStatus.
Resolves #1186. Check for a null pointer in ADAMContext.getFiles. Make
ADAMContext.getFsAndFiles package private so that we can unit test it.
Added unit tests to check for proper exception with empty directories and
bad globs.
@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Sep 28, 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/1506/
Test PASSed.

AmplabJenkins commented Sep 28, 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/1506/
Test PASSed.

@heuermh heuermh merged commit f4a33a5 into bigdatagenomics:master Sep 28, 2016

1 check passed

default Merged build finished.
Details
@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Sep 28, 2016

Member

Thank you, @fnothaft!

Member

heuermh commented Sep 28, 2016

Thank you, @fnothaft!

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