Add graphml output to CommandLineApp and DomainGraphExtractor.

[ruebot]

GitHub issue(s): #435

What does this Pull Request do?

Add graphml output to CommandLineApp and DomainGraphExtractor.

• Resolves Add graphml output to DomainGraphExtractor #435
• Adds GRAPHML option to CommandLineApp
• Add DataFrame method to DomainGraphExtractor
• Updates CommandLineApp test

How should this be tested?

TravisCI +

bin/spark-submit --class io.archivesunleashed.app.CommandLineAppRunner /home/nruest/Projects/au/aut/target/aut-0.50.1-SNAPSHOT-fatjar.jar --extractor DomainGraphExtractor --input /home/nruest/Projects/au/sample-data/geocities/* --output /home/nruest/Projects/au/sample-data/app-output/graphml-rdd --output-format GRAPHML

bin/spark-submit --class io.archivesunleashed.app.CommandLineAppRunner /home/nruest/Projects/au/aut/target/aut-0.50.1-SNAPSHOT-fatjar.jar --extractor DomainGraphExtractor --input /home/nruest/Projects/au/sample-data/geocities/* --output /home/nruest/Projects/au/sample-data/app-output/graphml-df --df --output-format GRAPHML

Those both should have the same line count.

For example:

$ wc -l graphml-df/GRAPHML.xml graphml-rdd/GRAPHML.xml
  33911 graphml-df/GRAPHML.xml
  33911 graphml-rdd/GRAPHML.xml
  67822 total

Additional Notes:

I should add a test as well for WriteGraphmlDF similar to what we have for WriteGexf. So, don't merge this right away, but feel free to test it and make sure it works.

This might also take care of the remainder of #223 and supersede #397? 👋 @SinghGursimran

[codecov]

Codecov Report

Merging #438 into master will increase coverage by 0.13%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master     #438      +/-   ##
==========================================
+ Coverage   78.04%   78.18%   +0.13%     
==========================================
  Files          43       43              
  Lines        1558     1586      +28     
  Branches      286      289       +3     
==========================================
+ Hits         1216     1240      +24     
- Misses        217      218       +1     
- Partials      125      128       +3     

[ianmilligan1]

See 🤦 comment from me below (but the file outputs are great!).

[ianmilligan1]

👍 Looks great on all fronts! (give me a thumbs up if you're happy to have this PR merged, @ruebot, looks good to me)

[ruebot]

@ianmilligan1 sure, let's use the following as a commit message:

Add graphml output to CommandLineApp and DomainGraphExtractor.

• Resolves Add graphml output to DomainGraphExtractor #435
• Adds GRAPHML option to CommandLineApp
• Adds DataFrame method to DomainGraphExtractor
• Updates CommandLineApp, and WriteGraphML tests