tkhurana commented on code in PR #2154:
URL: https://github.com/apache/phoenix/pull/2154#discussion_r2093337298


##########
phoenix-core-client/src/main/java/org/apache/phoenix/util/IndexUtil.java:
##########
@@ -856,6 +856,40 @@ public static void removeEmptyColumn(Mutation m, byte[] 
emptyCF, byte[] emptyCQ)
         }
     }
 
+    /**
+     * Creates a true deep copy of the Put Mutation, including deep copies of 
all cells if the
+     * cells are backed by off-heap. The Mutation(Mutation source) constructor 
always does a
+     * shallow copy of the cells.
+     * @param original The original Put Mutation to copy
+     * @return A new Put Mutation with deep copies of all fields and Cells
+     */
+    public static Put copyPut(Put original) throws IOException {

Review Comment:
   @virajjasani How about `MutationUtil` ? 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to