[
https://issues.apache.org/jira/browse/RATIS-691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16953418#comment-16953418
]
Hadoop QA commented on RATIS-691:
---------------------------------
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m
18s{color} | {color:blue} Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue} 0m
0s{color} | {color:blue} Findbugs executables are not available. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m
0s{color} | {color:green} The patch appears to include 1 new or modified test
files. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 3m
3s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m
53s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m
21s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m
44s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m
59s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m
52s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m
52s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m
11s{color} | {color:green} root: The patch generated 0 new + 0 unchanged - 81
fixed = 0 total (was 81) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m
0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m
34s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:red}-1{color} | {color:red} unit {color} | {color:red} 17m 23s{color}
| {color:red} root in the patch failed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m
17s{color} | {color:green} The patch does not generate ASF License warnings.
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 25m 53s{color} |
{color:black} {color} |
\\
\\
|| Reason || Tests ||
| Failed junit tests | ratis.logservice.TestLogServiceWithNetty |
| | ratis.logservice.TestLogServiceWithGrpc |
| | ratis.logservice.server.TestMetaServer |
| | ratis.grpc.TestServerRestartWithGrpc |
| | ratis.server.simulation.TestRaftReconfigurationWithSimulatedRpc |
| | ratis.server.raftlog.TestRaftLogMetrics |
| | ratis.grpc.TestWatchRequestWithGrpc |
| | ratis.grpc.TestLeaderElectionWithGrpc |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=19.03.3 Server=19.03.3 Image:yetus/ratis:date2019-10-17 |
| JIRA Issue | RATIS-691 |
| JIRA Patch URL |
https://issues.apache.org/jira/secure/attachment/12983227/RATIS-691.003.patch |
| Optional Tests | dupname asflicense javac javadoc unit findbugs
checkstyle compile |
| uname | Linux 3be5da092684 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24
10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality |
/home/jenkins/jenkins-slave/workspace/PreCommit-RATIS-Build/yetus-personality.sh
|
| git revision | master / b931482 |
| maven | version: Apache Maven 3.6.2
(40f52333136460af0dc0d7232c0dc0bcf0d9e117; 2019-08-27T15:06:16Z) |
| Default Java | 1.8.0_222 |
| unit |
https://builds.apache.org/job/PreCommit-RATIS-Build/1072/artifact/out/patch-unit-root.txt
|
| Test Results |
https://builds.apache.org/job/PreCommit-RATIS-Build/1072/testReport/ |
| Max. process+thread count | 2002 (vs. ulimit of 5000) |
| modules | C: ratis-logservice U: ratis-logservice |
| Console output |
https://builds.apache.org/job/PreCommit-RATIS-Build/1072/console |
| Powered by | Apache Yetus 0.8.0 http://yetus.apache.org |
This message was automatically generated.
> Fix checkstyle violations in ratis-logservice
> ---------------------------------------------
>
> Key: RATIS-691
> URL: https://issues.apache.org/jira/browse/RATIS-691
> Project: Ratis
> Issue Type: Sub-task
> Components: LogService
> Reporter: Dinesh Chitlangia
> Assignee: Dinesh Chitlangia
> Priority: Minor
> Attachments: RATIS-691.001.patch, RATIS-691.002.patch,
> RATIS-691.003.patch
>
> Time Spent: 1h
> Remaining Estimate: 0h
>
> Checkstyle found 83 item(s) in 22 file(s)
> * ArchiveHdfsLogReader.java : 3 item(s)
> ** Unused import - java.io.File. (21:8) [UnusedImportsCheck]
> ** Unused import - java.nio.Buffer. (24:8) [UnusedImportsCheck]
> ** Unused import - org.apache.ratis.logservice.api.LogName. (39:8)
> [UnusedImportsCheck]
> * ArchiveHdfsLogWriter.java : 1 item(s)
> ** Unused import - org.apache.ratis.util.LogUtils. (32:8)
> [UnusedImportsCheck]
> * ArchivedLogStreamImpl.java : 3 item(s)
> ** Variable 'name' must be private and have accessor methods. (45:11)
> [VisibilityModifierCheck]
> ** Variable 'config' must be private and have accessor methods. (49:27)
> [VisibilityModifierCheck]
> ** Variable 'state' must be private and have accessor methods. (53:9)
> [VisibilityModifierCheck]
> * CommandFactory.java : 1 item(s)
> ** Class CommandFactory should be declared as final. (35:0) [FinalClassCheck]
> * Constants.java : 1 item(s)
> ** Utility classes should not have a public or default constructor. (25:1)
> [HideUtilityClassConstructorCheck]
> * LogMessage.java : 1 item(s)
> ** Variable 'logName' must be private and have accessor methods. (27:21)
> [VisibilityModifierCheck]
> * LogName.java : 1 item(s)
> ** Class LogName should be declared as final. (31:0) [FinalClassCheck]
> * LogReaderImpl.java : 2 item(s)
> ** Variable 'currentRecordId' must be private and have accessor methods.
> (61:8) [VisibilityModifierCheck]
> ** File contains tab characters (this is the first instance). (126:1)
> [FileTabCharacterCheck]
> * LogServer.java : 2 item(s)
> ** Variable 'peer' must be private and have accessor methods. (192:18)
> [VisibilityModifierCheck]
> ** Redundant 'public' modifier. (193:9) [RedundantModifierCheck]
> * LogServiceClient.java : 3 item(s)
> ** 'private' modifier out of order with the JLS suggestions. (57:11)
> [ModifierOrderCheck]
> ** 'private' modifier out of order with the JLS suggestions. (58:11)
> [ModifierOrderCheck]
> ** 'config' hides a field. (269:68) [HiddenFieldCheck]
> * LogServiceMetricsRegistry.java : 2 item(s)
> ** Utility classes should not have a public or default constructor. (28:1)
> [HideUtilityClassConstructorCheck]
> ** Variable 'metricsReporting' must be private and have accessor methods.
> (35:27) [VisibilityModifierCheck]
> * LogServiceShell.java : 1 item(s)
> ** Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. (46:25)
> [StaticVariableNameCheck]
> * LogServiceShellOpts.java : 1 item(s)
> ** Variable 'metaQuorum' must be private and have accessor methods. (24:17)
> [VisibilityModifierCheck]
> * LogServiceUtils.java : 1 item(s)
> ** Utility classes should not have a public or default constructor. (32:1)
> [HideUtilityClassConstructorCheck]
> * LogStateMachine.java : 13 item(s)
> ** 'state' hides a field. (185:19) [HiddenFieldCheck]
> ** Redundant 'final' modifier. (199:9) [RedundantModifierCheck]
> ** Redundant 'final' modifier. (206:9) [RedundantModifierCheck]
> ** Redundant 'final' modifier. (207:9) [RedundantModifierCheck]
> ** Redundant 'final' modifier. (236:9) [RedundantModifierCheck]
> ** Redundant 'final' modifier. (237:9) [RedundantModifierCheck]
> ** 'archivalInfo' hides a field. (334:9) [HiddenFieldCheck]
> ** '{' at column 3 should be on the previous line. (347:3) [LeftCurlyCheck]
> ** '{' at column 3 should be on the previous line. (362:3) [LeftCurlyCheck]
> ** Redundant 'final' modifier. (464:12) [RedundantModifierCheck]
> ** switch without "default" clause. (538:0) [MissingSwitchDefaultCheck]
> ** 'state' hides a field. (576:16) [HiddenFieldCheck]
> ** 'state' hides a field. (747:45) [HiddenFieldCheck]
> * LogStream.java : 1 item(s)
> ** Redundant 'public' modifier. (33:3) [RedundantModifierCheck]
> * LogStreamImpl.java : 6 item(s)
> ** Variable 'listeners' must be private and have accessor methods. (51:24)
> [VisibilityModifierCheck]
> ** Variable 'name' must be private and have accessor methods. (55:11)
> [VisibilityModifierCheck]
> ** Variable 'raftClient' must be private and have accessor methods. (59:14)
> [VisibilityModifierCheck]
> ** Variable 'config' must be private and have accessor methods. (63:27)
> [VisibilityModifierCheck]
> ** Variable 'state' must be private and have accessor methods. (67:19)
> [VisibilityModifierCheck]
> ** Variable 'length' must be private and have accessor methods. (72:8)
> [VisibilityModifierCheck]
> * MetaServiceProtoUtil.java : 3 item(s)
> ** Unused import - org.apache.ratis.proto.RaftProtos. (27:8)
> [UnusedImportsCheck]
> ** Utility classes should not have a public or default constructor. (40:1)
> [HideUtilityClassConstructorCheck]
> ** Variable 'i' must be private and have accessor methods. (192:38)
> [VisibilityModifierCheck]
> * MetaStateMachine.java : 18 item(s)
> ** Unused import - org.apache.ratis.metrics.impl.RatisMetricRegistryImpl.
> (53:8) [UnusedImportsCheck]
> ** Name 'LOG' must match pattern '^[a-z][a-zA-Z0-9]*$'. (78:12)
> [MemberNameCheck]
> ** Variable 'LOG' must be private and have accessor methods. (78:12)
> [VisibilityModifierCheck]
> ** Must have at least one statement. (163:43) [EmptyBlockCheck]
> ** 'properties' hides a field. (218:28) [HiddenFieldCheck]
> ** 'peers' hides a field. (260:34) [HiddenFieldCheck]
> ** Line is longer than 120 characters (found 124). (294:0) [LineLengthCheck]
> ** Redundant 'final' modifier. (315:14) [RedundantModifierCheck]
> ** Line is longer than 120 characters (found 124). (331:0) [LineLengthCheck]
> ** 'peers' hides a field. (332:32) [HiddenFieldCheck]
> ** Variable 'peer' must be private and have accessor methods. (402:18)
> [VisibilityModifierCheck]
> ** Variable 'groups' must be private and have accessor methods. (403:24)
> [VisibilityModifierCheck]
> ** Redundant 'public' modifier. (405:9) [RedundantModifierCheck]
> ** Line is longer than 120 characters (found 123). (437:0) [LineLengthCheck]
> ** Line is longer than 120 characters (found 130). (456:0) [LineLengthCheck]
> ** 'if' construct must use '{}'s. (496:0) [NeedBracesCheck]
> ** 'if' construct must use '{}'s. (497:0) [NeedBracesCheck]
> ** 'if' construct must use '{}'s. (502:0) [NeedBracesCheck]
> * MetadataServer.java : 1 item(s)
> ** Variable 'metaStateMachine' must be private and have accessor methods.
> (54:18) [VisibilityModifierCheck]
> * RaftLogReader.java : 7 item(s)
> ** Unused import - org.apache.ratis.server.raftlog.RaftLogIOException.
> (22:8) [UnusedImportsCheck]
> ** Unused import -
> org.apache.ratis.thirdparty.com.google.protobuf.ByteString. (23:8)
> [UnusedImportsCheck]
> ** Unused import -
> org.apache.ratis.thirdparty.com.google.protobuf.InvalidProtocolBufferException.
> (24:8) [UnusedImportsCheck]
> ** Redundant 'public' modifier. (32:3) [RedundantModifierCheck]
> ** Redundant 'public' modifier. (37:3) [RedundantModifierCheck]
> ** Redundant 'public' modifier. (43:3) [RedundantModifierCheck]
> ** Redundant 'public' modifier. (49:3) [RedundantModifierCheck]
> * VerificationTool.java : 11 item(s)
> ** Line is longer than 120 characters (found 121). (71:0) [LineLengthCheck]
> ** Line is longer than 120 characters (found 144). (73:0) [LineLengthCheck]
> ** Line is longer than 120 characters (found 142). (79:0) [LineLengthCheck]
> ** Line is longer than 120 characters (found 162). (85:0) [LineLengthCheck]
> ** Line has trailing spaces. (96:0) [RegexpSinglelineCheck]
> ** 'abstract' modifier out of order with the JLS suggestions. (178:12)
> [ModifierOrderCheck]
> ** Variable 'logName' must be private and have accessor methods. (180:21)
> [VisibilityModifierCheck]
> ** Variable 'client' must be private and have accessor methods. (181:30)
> [VisibilityModifierCheck]
> ** Variable 'numRecords' must be private and have accessor methods. (182:17)
> [VisibilityModifierCheck]
> ** Variable 'logFreq' must be private and have accessor methods. (183:17)
> [VisibilityModifierCheck]
> ** Variable 'valueSize' must be private and have accessor methods. (184:17)
> [VisibilityModifierCheck]
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
