[ 
https://issues.apache.org/jira/browse/IGNITE-29094?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18124682#comment-18124682
 ] 

Ignite TC Bot commented on IGNITE-29094:
----------------------------------------

Docs Required was removed after reviewing the merged implementation in PR 
#13634. The change only removes an internal byte-array allocation in the thin 
JDBC request path (`JdbcThinTcpIo`) and does not add or change a user-facing 
API, configuration option, command, metric, documented behavior, or error 
message. Release Notes Required is preserved.

> Get rid of useless byte array creation during thin jdbc request
> ---------------------------------------------------------------
>
>                 Key: IGNITE-29094
>                 URL: https://issues.apache.org/jira/browse/IGNITE-29094
>             Project: Ignite
>          Issue Type: Improvement
>            Reporter: Nikolay Izhikov
>            Assignee: Nikolay Izhikov
>            Priority: Major
>              Labels: ise
>             Fix For: 2.19
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> Currently, useless byte array creation happens during each request.
>  
> {noformat}
>     /**
>      * @param req Request.
>      * @throws IOException In case of IO error.
>      */
>     private void sendRequestRaw(JdbcRequest req) throws IOException {
>         int cap = guessCapacity(req);
>         BinaryWriterEx writer = BinaryUtils.writer(ctx, 
> BinaryStreams.outputStream(cap));
>         req.writeBinary(writer, protoCtx);
>         synchronized (connMux) {
>             send(writer.out().arrayCopy());
>         }
>     }
> {noformat}
> We can easily simlify this 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to