hadoop-pig-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alan Gates (JIRA)" <j...@apache.org>
Subject [jira] Updated: (PIG-106) Optimize Pig by replacing String '+' and StringBuffer with StringBuilder
Date Mon, 17 Mar 2008 19:36:28 GMT

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

Alan Gates updated PIG-106:
---------------------------

       Resolution: Fixed
    Fix Version/s: 0.1.0
           Status: Resolved  (was: Patch Available)

Fix checked in.  Thanks Benjamin.

> Optimize Pig by replacing String '+' and StringBuffer with StringBuilder
> ------------------------------------------------------------------------
>
>                 Key: PIG-106
>                 URL: https://issues.apache.org/jira/browse/PIG-106
>             Project: Pig
>          Issue Type: Improvement
>          Components: impl
>    Affects Versions: 0.1.0
>            Reporter: Benjamin Francisoud
>            Assignee: Benjamin Francisoud
>             Fix For: 0.1.0
>
>         Attachments: PIG-106-v01.patch, PIG-106-v02.patch, PIG-106-v03.patch
>
>
> While investigating PIG-99, in TestBuiltin.java line 315:
> {code:java}
> for (int i = 0; i < LOOP_COUNT; i++) {
>     for (int j = 0; j < LOOP_COUNT; j++) {
>         sb.append(i + "\t" + i + "\t" + j % 2 + "\n");
>     }
> }
> {code}
> doing "i + "\t" + i + "\t" + j % 2 + "\n"" creates temporary String(s) reducing the advantages
of using a StringBuffer.
> Could be replace with:
> {code:java}
> for (int i = 0; i < LOOP_COUNT; i++) {
>     for (int j = 0; j < LOOP_COUNT; j++) {
>         sb.append(i);
>         sb.append("\t");
>         sb.append(i);
>         sb.append("\t");
>         sb.append(j % 2);
>         sb.append("\n");
>     }
> }
> {code}

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message