apache / apache/parquet-java

Use StringBuilder instead of StringBuffer

Offen
#2,530 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Component: Java Component: Parquet Priority: Trivial Type: enhancement
Vorherrschende Sprache
Java
Sterne
3.1k
Forks
1.6k
Ø Merge
3 T. 12 Std.
Gemergte PRs (30 T.)
33

Beschreibung

```java

private StringBuffer currentLine = new StringBuffer();

....

public String nextToken() {
while (st.hasMoreTokens()) {
String t = st.nextToken();
if (t.equals("\n")) {
++ line;
currentLine.setLength(0);
} else {
currentLine.append(t);
}
if (!isWhitespace(t)) {
return t;
}
}
throw new IllegalArgumentException("unexpected end of schema");
}
```

Use `StringBuilder` instead of `StringBuffer` as `StringBuffer` is synchronized (which is not required here).

**Reporter**: [David Mollitor](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=belugabehr) / @belugabehr

**Note**: *This issue was originally created as [PARQUET-1921](https://issues.apache.org/jira/browse/PARQUET-1921). Please see the [migration documentation](https://issues.apache.org/jira/browse/PARQUET-2502) for further details.*

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Durchsuche das Repository nach der gezeigten StringBuffer-Deklaration und der Verwendung von currentLine und lies anschließend den umgebenden Tokenizer-Code, um zu bestätigen, dass der Puffer nicht zwischen Threads gemeinsam verwendet wird. Ersetze die identifizierte Verwendung durch StringBuilder und führe den relevanten Java-Build- oder Testbefehl des Repositorys aus, um zu überprüfen, dass sich der Tokenizer weiterhin korrekt verhält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
data-engineering
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.