Skip to content

/ aut

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

Jump to bottom

Command line app fails because of missing log4j configuration #433

Closed
ruebot opened this issue Apr 7, 2020 · 1 comment
Closed

Command line app fails because of missing log4j configuration #433

ruebot opened this issue Apr 7, 2020 · 1 comment
Assignees
@ruebot
Labels
Scala bug
Projects
1.0.0 Release of AUT

Comments

@ruebot
Copy link
Member

@ruebot ruebot commented Apr 7, 2020

Describe the bug
Command line app fails because of missing log4j configuration

To Reproduce
Steps to reproduce the behavior (e.g.):

  1. bin/spark-submit --master local\[8\] ~/Projects/au/aut-releases/aut-0.50.0-fatjar.jar --extractor DomainFrequencyExtractor --input /home/nruest/Projects/au/sample-data/geocities/* --output /home/nruest/Projects/au/sample-data/app-output/DomainFrequencyExtractor

This will produce a misleading message when Spark fires up, then exit without running.

log4j:WARN No appenders could be found for logger (io.archivesunleashed.app.CommandLineApp).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See logging.apache.org/log4j/1.2/faq.html#noconfig for more info.

Environment information

  • AUT version: 0.50.0
  • OS: Ubuntu 18.04
  • Java version: Java 8
  • Apache Spark version: 2.4.5
  • Apache Spark w/aut: submit
  • Apache Spark command used to run AUT: ``bin/spark-submit --master local[8] ~/Projects/au/aut-releases/aut-0.50.0-fatjar.jar --extractor DomainFrequencyExtractor --input /home/nruest/Projects/au/sample-data/geocities/* --output /home/nruest/Projects/au/sample-data/app-output/DomainFrequencyExtractor

Additional context

One resolution here could be that we ship a sample log4j configuration like this in src/resources, and then document it.
@ruebot ruebot added bug Scala labels Apr 7, 2020
@ruebot ruebot self-assigned this Apr 7, 2020
@ruebot ruebot mentioned this issue Apr 7, 2020
Merged
@ruebot

This comment has been minimized.

Sign in to view
Copy link
Member Author

@ruebot ruebot commented Apr 7, 2020

🤦‍♂ we do ship one https://github.com/archivesunleashed/aut/blob/master/src/main/resources/log4j.properties
ruebot added a commit that referenced this issue Apr 7, 2020
@ruebot
Update log4j configuration to resolve #433.
Verified
This commit was signed with a verified signature.
ruebot Nick Ruest
GPG key ID: 417FAF1A0E1080CD Learn about signing commits
Loading status checks…
eed5a4f
@ruebot ruebot mentioned this issue Apr 7, 2020
Merged
@ianmilligan1 ianmilligan1 added this to Done in 1.0.0 Release of AUT via automation Apr 7, 2020
ianmilligan1 pushed a commit that referenced this issue Apr 7, 2020
@ruebot
Update log4j configuration to resolve #433. (#434)
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
b957e5d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ruebot ruebot

Labels
Scala bug
Projects
1.0.0 Release of AUT
  
Done
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

Update log4j configuration to resolve #433.
1 participant
@ruebot
You can’t perform that action at this time.