[
https://issues.apache.org/jira/browse/RATIS-8?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15949584#comment-15949584
]
Enis Soztutar commented on RATIS-8:
-----------------------------------
Patch looks good. However, can you also get rid of Optionals from the method
return signatures as well. {{TransactionContext}} is a core data structure,
which we are expected to be creating tens of thousands per second. Better to
get rid of extra object creations.
> 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)
