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
[NEMO-429] SWPP TEAM1 Code Smell Fix #267
Conversation
lambda to reference
Team 1, Daeeun Kim
Fix stream transient code smell
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.
Thanks for the PR!! but it looks like you missed some issues in https://sonarcloud.io/project/issues?fileUuids=AWkltI3qgtFwgfOLu2Ep%2CAWkltI3rgtFwgfOLu2It&id=org.apache.nemo%3Anemo-project&resolved=false&rules=squid%3AS1192%2Csquid%3AS1602%2Csquid%3AS1612
(Remove useless curly braces around statement in Util.java)
Could you please fix the issues and update this PR?
@@ -59,6 +59,7 @@ public String toString() { | |||
private final class NemoEventDecoder implements DecoderFactory.Decoder { | |||
|
|||
private final Decoder valueDecoder; | |||
//NOSONAR |
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.
please remove NOSONAR
@@ -58,6 +58,7 @@ public String toString() { | |||
*/ | |||
private final class NemoEventEncoder<T> implements EncoderFactory.Encoder<T> { | |||
private final EncoderFactory.Encoder<T> valueEncoder; | |||
//NOSONAR |
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.
please remove NOSONAR
remove useless curly braces
Revert "Fix stream transient code smell"
done |
LGTM. I'm going to merge it. Thanks! |
JIRA: NEMO-429: Dec 2nd, 2019 Code session
Major changes:
Fixed code smells