Skip to content

Commit b4fcd51

Browse files
authored
Make TestRateLimiter locale-independent (#469)
The vendored Guava test formatted expected events with the default locale, so it failed on JVMs using a comma decimal separator. Assisted-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 10d64e8 commit b4fcd51

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

flume-ng-core/src/test/java/org/apache/flume/source/shaded/guava/TestRateLimiter.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import com.google.common.collect.Lists;
2424
import java.util.Arrays;
2525
import java.util.List;
26+
import java.util.Locale;
2627
import java.util.Random;
2728
import junit.framework.TestCase;
2829
import org.apache.flume.source.shaded.guava.RateLimiter.SleepingStopwatch;
@@ -428,7 +429,7 @@ void sleepMillis(int millis) {
428429

429430
void sleepMicros(String caption, long micros) {
430431
instant += MICROSECONDS.toNanos(micros);
431-
events.add(caption + String.format("%3.2f", (micros / 1000000.0)));
432+
events.add(caption + String.format(Locale.ROOT, "%3.2f", (micros / 1000000.0)));
432433
}
433434

434435
@Override

0 commit comments

Comments
 (0)