[ADAM-1280] Silence CRAM logging in tests. #1294

Merged
merged 1 commit into from Nov 28, 2016

Conversation

Projects
None yet
4 participants
@fnothaft
Member

fnothaft commented Nov 28, 2016

Resolves #1280.

I tried for a while but couldn't silence the Parquet logging in the tests. We're still on Parquet 1.8.1, which messes with the Java util logging package, while Parquet 1.9.0 upgrades to Log4j. I figured that we'll punt on silencing the Parquet logs until we upgrade to 1.9.0, since that should then be a trivial fix.

@AmplabJenkins

This comment has been minimized.

Show comment
Hide comment
@AmplabJenkins

AmplabJenkins Nov 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/1641/
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/1641/
Test PASSed.

@heuermh

Thank you! Much appreciated

@heuermh heuermh merged commit 4c8ef3b into bigdatagenomics:master Nov 28, 2016

1 check passed

default Merged build finished.
Details
@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Nov 28, 2016

Member

Thank you, @fnothaft!

Member

heuermh commented Nov 28, 2016

Thank you, @fnothaft!

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Jan 23, 2017

Member

The 1.8.2 parquet release may help (https://issues.apache.org/jira/browse/PARQUET-423).
@rdblue I don't see the 1.8.2 release in Maven Central, is it just waiting on the rsync?

Member

heuermh commented Jan 23, 2017

The 1.8.2 parquet release may help (https://issues.apache.org/jira/browse/PARQUET-423).
@rdblue I don't see the 1.8.2 release in Maven Central, is it just waiting on the rsync?

@rdblue

This comment has been minimized.

Show comment
Hide comment
@rdblue

rdblue Jan 23, 2017

The 1.8.2 release is still a candidate, we're voting on it right now. PARQUET-423 is included in that release.

rdblue commented Jan 23, 2017

The 1.8.2 release is still a candidate, we're voting on it right now. PARQUET-423 is included in that release.

@heuermh

This comment has been minimized.

Show comment
Hide comment
@heuermh

heuermh Jan 23, 2017

Member

Got it, thank you, @rdblue! Forgot about the Apache release process, I'll drop in on the dev@ mailing list to check out the release candidate.

Member

heuermh commented Jan 23, 2017

Got it, thank you, @rdblue! Forgot about the Apache release process, I'll drop in on the dev@ mailing list to check out the release candidate.

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