scw00 commented on a change in pull request #6961:
URL: https://github.com/apache/trafficserver/pull/6961#discussion_r446727853



##########
File path: iocore/cache/CacheWrite.cc
##########
@@ -617,7 +617,6 @@ Vol::evacuateDocReadDone(int event, Event *e)
   if (doc->magic != DOC_MAGIC) {
     Debug("cache_evac", "DOC magic: %X %d", 
(int)dir_tag(&doc_evacuator->overwrite_dir),

Review comment:
       Because it is unexpected behavior. It should not happen in normal 
processing. We should collect more information instead of throw a debug log. 
   
   On the other way, IIUC, the data will be truncated if we goto Ldone.




----------------------------------------------------------------
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:
[email protected]


Reply via email to