[jira] [Commented] (TEZ-3907) Improve log message to include the location the writers decide to spill output

["Jason Lowe (JIRA)" <ji...@apache.org>]

    [ 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)