[
https://issues.apache.org/jira/browse/RATIS-8?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15948538#comment-15948538
]
ASF GitHub Bot commented on RATIS-8:
------------------------------------
GitHub user hmcl opened a pull request:
https://github.com/apache/incubator-ratis/pull/1
RATIS-8: Avoid using Optional for data fields in TransactionContex
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/hmcl/incubator-ratis RATIS-8_Optional
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-ratis/pull/1.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1
----
commit 290369e244eaf5504f471c01812bc1f18eca981b
Author: Hugo Louro <[email protected]>
Date: 2017-03-30T05:54:44Z
RATIS-8: Avoid using Optional for data fields in TransactionContex
----
> Avoid using Optional for data fields in TransactionContext
> ----------------------------------------------------------
>
> Key: RATIS-8
> URL: https://issues.apache.org/jira/browse/RATIS-8
> Project: Ratis
> Issue Type: Improvement
> Reporter: Jing Zhao
> Assignee: Hugo Louro
> Attachments: RATIS-8.patch
>
>
> It's usually not recommended to use {{Optional}} as class fields. Let's use
> this jira to revisit the {{Optional}} usage in {{TransactionContext}}.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
