accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <>
Subject [jira] [Updated] (ACCUMULO-2663) Remove usage of String#split from Combiner#init
Date Fri, 13 Jun 2014 01:35:01 GMT


Josh Elser updated ACCUMULO-2663:

    Resolution: Fixed
        Status: Resolved  (was: Patch Available)

Thanks so much for the patch, [~dmprotz]! I applied this to all active branches since it was
very simple.

Please let me know if you'd like to be listed on our [contributor's list|],
optionally including timezone and/or employer details.

> Remove usage of String#split from Combiner#init
> -----------------------------------------------
>                 Key: ACCUMULO-2663
>                 URL:
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Josh Elser
>            Assignee: David Protzman
>            Priority: Minor
>              Labels: newbie, summit2014
>             Fix For: 1.5.2, 1.6.1, 1.7.0
>         Attachments: ACCUMULO-2663.v1.patch
> Noticed that inside {{Combiner#init}}, the comma-separated list of columns is split out
using the inefficient {{String#split}} method. This should be switched into something that
doesn't used Regexs since this method is invoked a surprising amount in normal usage.

This message was sent by Atlassian JIRA

View raw message