[
https://issues.apache.org/jira/browse/TEZ-3379?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15395996#comment-15395996
]
Hitesh Shah commented on TEZ-3379:
----------------------------------
+1. Minor nit: for slf4j, it is better to use "{}" based placeholders to avoid
string creation. In any case, given that this is a command line tool without
much perf impact, this review comment is optional to address.
> Tez analyzer: Move sysout to log4j
> ----------------------------------
>
> Key: TEZ-3379
> URL: https://issues.apache.org/jira/browse/TEZ-3379
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Rajesh Balamohan
> Assignee: Rajesh Balamohan
> Priority: Trivial
> Attachments: TEZ-3379.1.patch, TEZ-3379.2.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
