honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Use StringBuilder in favor of StringBuffer 

This removes all instances of StringBuffer that are removeable.

Uncontended synchronization in Java is pretty cheap, but it's unnecessary.
This commit is contained in:
Chris Earle 2016-08-17 17:36:46 -04:00
parent b41508a344
commit 1cf694b63e
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
core/src/main/java/org/elasticsearch/env/ShardLockObtainFailedException.java vendored
View File

@ -39,7 +39,7 @@ public class ShardLockObtainFailedException extends Exception {
@Override @Override
public String getMessage() { public String getMessage() {
StringBuffer sb = new StringBuffer(); StringBuilder sb = new StringBuilder();
sb.append(shardId.toString()); sb.append(shardId.toString());
sb.append(": "); sb.append(": ");
sb.append(super.getMessage()); sb.append(super.getMessage());

2
plugins/analysis-phonetic/src/main/java/org/elasticsearch/index/analysis/phonetic/Nysiis.java
View File

@ -263,7 +263,7 @@ public class Nysiis implements StringEncoder {
str = PAT_DT_ETC.matcher(str).replaceFirst("D"); str = PAT_DT_ETC.matcher(str).replaceFirst("D");
// First character of key = first character of name. // First character of key = first character of name.
StringBuffer key = new StringBuffer(str.length()); StringBuilder key = new StringBuilder(str.length());
key.append(str.charAt(0)); key.append(str.charAt(0));
// Transcode remaining characters, incrementing by one character each time // Transcode remaining characters, incrementing by one character each time