[ADAM-1099] Make reference region not throw NPE. #1115

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@fnothaft
Member

fnothaft commented Aug 13, 2016

Resolves #1099.

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Aug 13, 2016

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

Build result: FAILURE

GitHub pull request #1115 of commit 7dd3022 automatically merged.Notifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos spark-test) in workspace /home/jenkins/workspace/ADAM-prb > /home/jenkins/git2/bin/git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > /home/jenkins/git2/bin/git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > /home/jenkins/git2/bin/git --version # timeout=10 > /home/jenkins/git2/bin/git -c core.askpass=true fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ # timeout=15 > /home/jenkins/git2/bin/git rev-parse origin/pr/1115/merge^{commit} # timeout=10 > /home/jenkins/git2/bin/git branch -a --contains b8db4a8f7c5f130ec789458db1554bcb941a288d # timeout=10 > /home/jenkins/git2/bin/git rev-parse remotes/origin/pr/1115/merge^{commit} # timeout=10Checking out Revision b8db4a8f7c5f130ec789458db1554bcb941a288d (origin/pr/1115/merge) > /home/jenkins/git2/bin/git config core.sparsecheckout # timeout=10 > /home/jenkins/git2/bin/git checkout -f b8db4a8f7c5f130ec789458db1554bcb941a288dFirst time build. Skipping changelog.Triggering ADAM-prb ? 2.6.0,2.10,1.5.2,centosTriggering ADAM-prb ? 2.6.0,2.11,1.5.2,centosTouchstone configurations resulted in FAILURE, so aborting...Notifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'
Test FAILed.

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

Build result: FAILURE

GitHub pull request #1115 of commit 7dd3022 automatically merged.Notifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos spark-test) in workspace /home/jenkins/workspace/ADAM-prb > /home/jenkins/git2/bin/git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > /home/jenkins/git2/bin/git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > /home/jenkins/git2/bin/git --version # timeout=10 > /home/jenkins/git2/bin/git -c core.askpass=true fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ # timeout=15 > /home/jenkins/git2/bin/git rev-parse origin/pr/1115/merge^{commit} # timeout=10 > /home/jenkins/git2/bin/git branch -a --contains b8db4a8f7c5f130ec789458db1554bcb941a288d # timeout=10 > /home/jenkins/git2/bin/git rev-parse remotes/origin/pr/1115/merge^{commit} # timeout=10Checking out Revision b8db4a8f7c5f130ec789458db1554bcb941a288d (origin/pr/1115/merge) > /home/jenkins/git2/bin/git config core.sparsecheckout # timeout=10 > /home/jenkins/git2/bin/git checkout -f b8db4a8f7c5f130ec789458db1554bcb941a288dFirst time build. Skipping changelog.Triggering ADAM-prb ? 2.6.0,2.10,1.5.2,centosTriggering ADAM-prb ? 2.6.0,2.11,1.5.2,centosTouchstone configurations resulted in FAILURE, so aborting...Notifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'
Test FAILed.

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Aug 14, 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/1386/
Test PASSed.

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

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Aug 16, 2016

Member

LGTM

Member

heuermh commented Aug 16, 2016

LGTM

@fnothaft

This comment has been minimized.

Show comment
Hide comment
@fnothaft

fnothaft Aug 25, 2016

Member

This is a trivial change; can I get this merged?

Member

fnothaft commented Aug 25, 2016

This is a trivial change; can I get this merged?

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Aug 25, 2016

Member

Merged as commit ca05a86

Thank you, @fnothaft!

Member

heuermh commented Aug 25, 2016

Merged as commit ca05a86

Thank you, @fnothaft!

@heuermh heuermh closed this Aug 25, 2016

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