[ https://issues.apache.org/jira/browse/HBASE-11308?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
stack updated HBASE-11308: -------------------------- Priority: Trivial (was: Minor) Definetly not in the critical path.... Suggest you go make code changes that will make an actual difference perf or otherwise. > Reuse NameStringPair#Builder instances in loops > ----------------------------------------------- > > Key: HBASE-11308 > URL: https://issues.apache.org/jira/browse/HBASE-11308 > Project: HBase > Issue Type: Task > Reporter: Ted Yu > Priority: Trivial > > The following code is from HColumnDescriptor#convert() but the pattern can be > found in other classes, such as HBaseAdmin: > {code} > for (Map.Entry<String, String> e : this.configuration.entrySet()) { > NameStringPair.Builder aBuilder = NameStringPair.newBuilder(); > aBuilder.setName(e.getKey()); > aBuilder.setValue(e.getValue()); > builder.addConfiguration(aBuilder.build()); > } > {code} > NameStringPair.Builder instance can be reused across iterations of the loop. > This would save creation of short-lived objects. -- This message was sent by Atlassian JIRA (v6.2#6252)