[
https://issues.apache.org/jira/browse/RATIS-62?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jing Zhao resolved RATIS-62.
Resolution: Fixed
Fix Version/s: 0.1.0-alpha
Thanks for the review, Mingliang! I've committed the
[
https://issues.apache.org/jira/browse/RATIS-62?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15951717#comment-15951717
]
Mingliang Liu commented on RATIS-62:
+1
> Return the Exception from StateMachine#preAppendTransaction
[
https://issues.apache.org/jira/browse/RATIS-62?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jing Zhao updated RATIS-62:
---
Attachment: RATIS-62.000.patch
Upload the patch.
> Return the Exception from StateMachine#preAppendTransaction
[
https://issues.apache.org/jira/browse/RATIS-62?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jing Zhao reassigned RATIS-62:
--
Assignee: Jing Zhao
> Return the Exception from StateMachine#preAppendTransaction to client as
>
[
https://issues.apache.org/jira/browse/RATIS-62?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jing Zhao updated RATIS-62:
---
Summary: Return the Exception from StateMachine#preAppendTransaction to
client as StateMachineException (was:
Jing Zhao created RATIS-62:
--
Summary: Wrap the Exception thrown by
StateMachine#preAppendTransaction as StateMachineException and return the
exception in RaftClientReply
Key: RATIS-62
URL:
[
https://issues.apache.org/jira/browse/RATIS-52?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15951388#comment-15951388
]
Enis Soztutar commented on RATIS-52:
bq. I think for this patch we can do the code refactoring work
[
https://issues.apache.org/jira/browse/RATIS-10?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jing Zhao resolved RATIS-10.
Resolution: Fixed
Fix Version/s: 0.1.0-alpha
Most of the work has been done for the retry cache. We
[
https://issues.apache.org/jira/browse/RATIS-17?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jing Zhao resolved RATIS-17.
Resolution: Fixed
Fix Version/s: 0.1.0-alpha
I've committed this.
> Add basic retry cache
[
https://issues.apache.org/jira/browse/RATIS-17?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950439#comment-15950439
]
Jing Zhao commented on RATIS-17:
Thanks for the review, Nicholas! Since RATIS-52 got reopened, I will commit
[
https://issues.apache.org/jira/browse/RATIS-52?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jing Zhao updated RATIS-52:
---
Attachment: ratis-52-fix.patch
The bug was caused by the newly added methods in ReflectionUtils.java, and it's
[
https://issues.apache.org/jira/browse/RATIS-52?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950424#comment-15950424
]
Jing Zhao edited comment on RATIS-52 at 3/31/17 7:05 AM:
-
Ahh, I just fixed it...
[
https://issues.apache.org/jira/browse/RATIS-52?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950424#comment-15950424
]
Jing Zhao commented on RATIS-52:
Ahh, I just fixed it... haha
I will posted my change here and maybe it's
[
https://issues.apache.org/jira/browse/RATIS-52?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Tsz Wo Nicholas Sze reopened RATIS-52:
--
Sorry, I have tried to fix it but it is not very easy. I have reverted it.
> Refactor
[
https://issues.apache.org/jira/browse/RATIS-54?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950404#comment-15950404
]
Tsz Wo Nicholas Sze commented on RATIS-54:
--
Thanks for the review,
> requiring the latest version
[
https://issues.apache.org/jira/browse/RATIS-52?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950396#comment-15950396
]
Jing Zhao commented on RATIS-52:
Looks like the patch breaks tests:
{code}
[
https://issues.apache.org/jira/browse/RATIS-17?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15950372#comment-15950372
]
Tsz Wo Nicholas Sze commented on RATIS-17:
--
+1 the 006 patch looks good. Please update it with
17 matches
Mail list logo