[jira] [Commented] (RATIS-303) TestRaftStateMachineException is failing with NullPointerException

2018-09-06 Thread Hadoop QA (JIRA)


[ 
https://issues.apache.org/jira/browse/RATIS-303?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16605635#comment-16605635
 ] 

Hadoop QA commented on RATIS-303:
-

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
21s{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 7 new or modified test 
files. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  1m 
29s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  2m 
33s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 
13s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
18s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
37s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m  
5s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  1m 
15s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m  
5s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  1m  
5s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
14s{color} | {color:green} the patch passed {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 
31s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:red}-1{color} | {color:red} unit {color} | {color:red} 15m 46s{color} 
| {color:red} root in the patch failed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
14s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 25m 48s{color} | 
{color:black} {color} |
\\
\\
|| Reason || Tests ||
| Failed junit tests | ratis.TestRaftServerSlownessDetection |
|   | ratis.server.simulation.TestRaftWithSimulatedRpc |
|   | ratis.TestRaftServerLeaderElectionTimeout |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/ratis:date2018-09-06 
|
| JIRA Issue | RATIS-303 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12938621/RATIS-303.02.patch |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  checkstyle  
compile  |
| uname | Linux 66e86d8c4138 3.13.0-143-generic #192-Ubuntu SMP Tue Feb 27 
10:45:36 UTC 2018 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 / acd507e |
| Default Java | 1.8.0_181 |
| unit | 
https://builds.apache.org/job/PreCommit-RATIS-Build/325/artifact/out/patch-unit-root.txt
 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-RATIS-Build/325/testReport/ |
| modules | C: ratis-server ratis-grpc ratis-netty ratis-hadoop ratis-examples 
U: . |
| Console output | 
https://builds.apache.org/job/PreCommit-RATIS-Build/325/console |
| Powered by | Apache Yetus 0.5.0   http://yetus.apache.org |


This message was automatically generated.



> TestRaftStateMachineException is failing with NullPointerException
> --
>
> Key: RATIS-303
> URL: https://issues.apache.org/jira/browse/RATIS-303
> Project: Ratis
>  Issue Type: Bug
>  Components: server
>Affects Versions: 0.3.0
>Reporter: Mukul Kumar Singh
>Assignee: Shashikant Banerjee
>Priority: Major
>  

[jira] [Commented] (RATIS-303) TestRaftStateMachineException is failing with NullPointerException

2018-09-06 Thread Shashikant Banerjee (JIRA)


[ 
https://issues.apache.org/jira/browse/RATIS-303?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16605535#comment-16605535
 ] 

Shashikant Banerjee commented on RATIS-303:
---

Thanks [~szetszwo], for the review. patch v1 moves 
TestRaftStateMachineException tests to ratis-server and adds the subclasses for 
the rpcs.

...Could you describe how the patch fix NullPointerException? 

The Null pointer exception was caused because in the TestStateMachine, a fake 
exception is thrown during preAppendTransaction. This leads to leader stepping 
down so subsequent client call with leader set to null fails with null pointer 
exception. Since the single cluster instance was shared among all the tests , 
leader being set to null intermittently lead to the failure of other tests as 
well.

The exception is addressed by waiting for the leader to come up and sending the 
next request to the proper leader after the stateMachine exception is thrown in 
testRetryOnExceptionDuringReplication.

> TestRaftStateMachineException is failing with NullPointerException
> --
>
> Key: RATIS-303
> URL: https://issues.apache.org/jira/browse/RATIS-303
> Project: Ratis
>  Issue Type: Bug
>  Components: server
>Affects Versions: 0.3.0
>Reporter: Mukul Kumar Singh
>Assignee: Shashikant Banerjee
>Priority: Major
>  Labels: ozone
> Fix For: 0.3.0
>
> Attachments: RATIS-303.02.patch
>
>
> TestRaftStateMachineException is failing with the following exception
> {code}
> [ERROR] 
> testRetryOnExceptionDuringReplication[2](org.apache.ratis.statemachine.TestRaftStateMachineException)
>   Time elapsed: 0.001 s  <<< ERROR!
> java.lang.NullPointerException
>   at 
> org.apache.ratis.statemachine.TestRaftStateMachineException.testRetryOnExceptionDuringReplication(TestRaftStateMachineException.java:139)
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (RATIS-303) TestRaftStateMachineException is failing with NullPointerException

2018-09-05 Thread Tsz Wo Nicholas Sze (JIRA)


[ 
https://issues.apache.org/jira/browse/RATIS-303?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16604878#comment-16604878
 ] 

Tsz Wo Nicholas Sze commented on RATIS-303:
---

Thanks [~shashikant].
- Could you describe how the patch fix NullPointerException?  It is not easy to 
see from the patch.
- For the new TestRetryOnStateMachineException, could you move it from 
ratis-example to ratis-server and add subclasses for the RPCs. See 
RaftExceptionBaseTest and its subclasses.  It was a mistake to put the 
non-example test in ratis-example.  It would be great if you could also move 
TestRaftStateMachineException.

> TestRaftStateMachineException is failing with NullPointerException
> --
>
> Key: RATIS-303
> URL: https://issues.apache.org/jira/browse/RATIS-303
> Project: Ratis
>  Issue Type: Bug
>  Components: server
>Affects Versions: 0.3.0
>Reporter: Mukul Kumar Singh
>Assignee: Shashikant Banerjee
>Priority: Major
>  Labels: ozone
> Fix For: 0.3.0
>
> Attachments: RATIS-303.01.patch
>
>
> TestRaftStateMachineException is failing with the following exception
> {code}
> [ERROR] 
> testRetryOnExceptionDuringReplication[2](org.apache.ratis.statemachine.TestRaftStateMachineException)
>   Time elapsed: 0.001 s  <<< ERROR!
> java.lang.NullPointerException
>   at 
> org.apache.ratis.statemachine.TestRaftStateMachineException.testRetryOnExceptionDuringReplication(TestRaftStateMachineException.java:139)
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (RATIS-303) TestRaftStateMachineException is failing with NullPointerException

2018-09-04 Thread Shashikant Banerjee (JIRA)


[ 
https://issues.apache.org/jira/browse/RATIS-303?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16602673#comment-16602673
 ] 

Shashikant Banerjee commented on RATIS-303:
---

Thanks [~msingh], for the review. patch v1 addresses your review comments.

> TestRaftStateMachineException is failing with NullPointerException
> --
>
> Key: RATIS-303
> URL: https://issues.apache.org/jira/browse/RATIS-303
> Project: Ratis
>  Issue Type: Bug
>  Components: server
>Affects Versions: 0.3.0
>Reporter: Mukul Kumar Singh
>Assignee: Shashikant Banerjee
>Priority: Major
>  Labels: ozone
> Fix For: 0.3.0
>
> Attachments: RATIS-303.01.patch
>
>
> TestRaftStateMachineException is failing with the following exception
> {code}
> [ERROR] 
> testRetryOnExceptionDuringReplication[2](org.apache.ratis.statemachine.TestRaftStateMachineException)
>   Time elapsed: 0.001 s  <<< ERROR!
> java.lang.NullPointerException
>   at 
> org.apache.ratis.statemachine.TestRaftStateMachineException.testRetryOnExceptionDuringReplication(TestRaftStateMachineException.java:139)
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)