hadoop-mapreduce-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Konstantin Weitz (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (MAPREDUCE-5708) Duplicate String.format in getSpillFileForWrite
Date Sat, 04 Jan 2014 18:09:51 GMT

     [ https://issues.apache.org/jira/browse/MAPREDUCE-5708?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Konstantin Weitz updated MAPREDUCE-5708:
----------------------------------------

    Status: Patch Available  (was: Open)

> Duplicate String.format in getSpillFileForWrite
> -----------------------------------------------
>
>                 Key: MAPREDUCE-5708
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-5708
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>            Reporter: Konstantin Weitz
>            Priority: Minor
>         Attachments: 0001-Removed-duplicate-String.format.patch
>
>   Original Estimate: 10m
>  Remaining Estimate: 10m
>
> The code responsible for formatting the spill file name (namely _getSpillFileForWrite_)
unnecessarily calls _String.format_ twice. This does not only affect performance, but leads
to the weird requirement that task attempt ids cannot contain _%_ characters (because these
would be interpreted as format specifiers in the outside _String.format_ call).
> I assume this was done by mistake, as it could only be useful if task attempt ids contained
_%n_.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Mime
View raw message