twitter / scalding

A Scala API for Cascading
http://twitter.com/scalding
Apache License 2.0
3.48k stars 704 forks source link

Clean-up scalding for 0.18.0 release #1891

Closed ttim closed 5 years ago

ttim commented 5 years ago

During 0.18.0-RC1 release I've found an issue with scalding-parquet-scrooge/src/main/java/com/twitter/scalding/parquet/scrooge/ScroogeReadSupport.java - it uses simple class name for nested class. ./sbt compile:doc goal fails in this case. I've fixed it to use full name (i.e. ReadSupport.ReadContext instead of ReadContext) and changed our scripts/run_test.sh script to prevent this type of issues in future.

I also included auto reformat changes to keep actual release clean.