anoopsjohn commented on pull request #3318:
URL: https://github.com/apache/hbase/pull/3318#issuecomment-850453676


   > I write a case how this error could happen in the doc: 
https://docs.google.com/document/d/1_3HXgOSGHsHFqLiOWUsE3m6pHKjebSwrRObcPTqkSTE/edit?usp=sharing,
 please have a look, thanks @Apache9 @anoopsjohn
   
   >The fix is simple, move the shipped method out of the for loop
   
   Ya the issue is very clear for me.  Actually in code, we clone the cells for 
which we keep the ref.   Seems missed in this place..  BTW in ur 1st patch that 
clone of Cell was there but removed now?  And instead did this move of shipped 
out of that for loop?


-- 
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.

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


Reply via email to