[
https://issues.apache.org/jira/browse/HDDS-886?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16704892#comment-16704892
]
Hudson commented on HDDS-886:
-----------------------------
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #15537 (See
[https://builds.apache.org/job/Hadoop-trunk-Commit/15537/])
HDDS-886. Unnecessary buffer copy in HddsDispatcher#dispatch. (yqlin: rev
62f821115be34f26e994790591c235710f0fc224)
* (edit)
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/impl/HddsDispatcher.java
> Unnecessary buffer copy in HddsDispatcher#dispatch
> --------------------------------------------------
>
> Key: HDDS-886
> URL: https://issues.apache.org/jira/browse/HDDS-886
> Project: Hadoop Distributed Data Store
> Issue Type: Bug
> Components: Ozone Datanode
> Reporter: Lokesh Jain
> Assignee: Lokesh Jain
> Priority: Major
> Fix For: 0.4.0
>
> Attachments: HDDS-886.001.patch
>
>
> In HddsDispatcher#dispatch precondition not null check converts container
> command to a string object. This is done even for a write chunk command which
> means we copy the chunk data to a string.
> {code:java}
> // code placeholderpublic ContainerCommandResponseProto dispatch(
> ContainerCommandRequestProto msg) {
> LOG.trace("Command {}, trace ID: {} ", msg.getCmdType().toString(),
> msg.getTraceID());
> Preconditions.checkNotNull(msg.toString());
> {code}
> The precondition needs to check only the msg.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
