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

Jason Lowe commented on TEZ-3907:
---------------------------------

Thanks for the patch!

Nit: " at :" s/b " at : " so the colon doesn't smash into the subsequent 
filename.  Actually I think it becomes more readable without the additional 
colon, both for this and the same log in UnorderedPartitionedKVWriter.

> Improve log message to include the location the writers decide to spill output
> ------------------------------------------------------------------------------
>
>                 Key: TEZ-3907
>                 URL: https://issues.apache.org/jira/browse/TEZ-3907
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Kuhu Shukla
>            Assignee: Kuhu Shukla
>            Priority: Major
>         Attachments: TEZ-3907.001.patch
>
>
> It helps debugging if the log message that prints the start of spill with the 
> buffer sizes and other pointers to include the location as well. This Jira 
> will add the already known location string to the log message.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to