DQ fix rope creation (#12073)

GrigoriyPA · web-flow · commit ddcb25a3478b · 2024-11-28T09:38:07.000+03:00
diff --git a/ydb/library/yql/dq/common/rope_over_buffer.cpp b/ydb/library/yql/dq/common/rope_over_buffer.cpp
@@ -37,7 +37,7 @@ TRope MakeReadOnlyRope(TChunkedBuffer&& buffer) {
     while (!buffer.Empty()) {
         auto& front = buffer.Front();
         TRope chunk(new TContigousChunkOverBuf(front.Owner, {front.Buf.data(), front.Buf.size()}));
-        result.Insert(result.End(), TRope(new TContigousChunkOverBuf(front.Owner, {front.Buf.data(), front.Buf.size()})));
+        result.Insert(result.End(), std::move(chunk));
         buffer.Erase(front.Buf.size());
     }
 

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ TRope MakeReadOnlyRope(TChunkedBuffer&& buffer) {`
`37`	`37`	`while (!buffer.Empty()) {`
`38`	`38`	`auto& front = buffer.Front();`
`39`	`39`	`TRope chunk(new TContigousChunkOverBuf(front.Owner, {front.Buf.data(), front.Buf.size()}));`
`40`		`- result.Insert(result.End(), TRope(new TContigousChunkOverBuf(front.Owner, {front.Buf.data(), front.Buf.size()})));`
	`40`	`+ result.Insert(result.End(), std::move(chunk));`
`41`	`41`	`buffer.Erase(front.Buf.size());`
`42`	`42`	`}`
`43`	`43`