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
warn combine*/sum* transform usage, fix #1620 #1856
Conversation
8311880
to
b7c31f3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there's some style issues and some minor comments but otherwise 👍
@@ -55,6 +56,8 @@ import scala.util.Try | |||
/** Convenience functions for creating SCollections. */ | |||
object SCollection { | |||
|
|||
private[values] val logger = LoggerFactory.getLogger(this.getClass) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
private[values] val logger = LoggerFactory.getLogger(this.getClass) | |
private[values] val Logger = LoggerFactory.getLogger(this.getClass) |
PairSCollectionFunctions
is also lower case can you change it?
Codecov Report
@@ Coverage Diff @@
## master #1856 +/- ##
==========================================
+ Coverage 73.12% 73.15% +0.03%
==========================================
Files 191 191
Lines 5823 5838 +15
Branches 430 437 +7
==========================================
+ Hits 4258 4271 +13
- Misses 1565 1567 +2
Continue to review full report at Codecov.
|
No description provided.