Changed to use StandardCharsets.UTF8.

Fixed potential NaN issue.
Changed limiter update logic to account for floating point variance.

Author: connelly38

driver/src/main/java/oracle/nosql/driver/util/RateLimiterMap.java

@@ -71,12 +71,18 @@ public synchronized void update(
             RateLimiter wrl = new SimpleRateLimiter(writeUnits, durationSeconds);
             limiterMap.put(lowerTable, new Entry(rrl, wrl));
         } else {
-            if (rle.readLimiter.getLimitPerSecond() != readUnits) {
-                rle.readLimiter.setLimitPerSecond(readUnits);
+            /*
+             * Set existing limiters to new values. If the new values result in a
+             * different rate than previous, reset the limiters.
+             */
+            double prevRUs = rle.readLimiter.getLimitPerSecond();
+            double prevWUs = rle.writeLimiter.getLimitPerSecond();
+            rle.readLimiter.setLimitPerSecond(readUnits);
+            rle.writeLimiter.setLimitPerSecond(writeUnits);
+            if (rle.readLimiter.getLimitPerSecond() != prevRUs) {
                 rle.readLimiter.reset();
             }
-            if (rle.writeLimiter.getLimitPerSecond() != writeUnits) {
-                rle.writeLimiter.setLimitPerSecond(writeUnits);
+            if (rle.writeLimiter.getLimitPerSecond() != prevWUs) {
                 rle.writeLimiter.reset();
             }
         }

driver/src/main/java/oracle/nosql/driver/util/SerializationUtil.java

@@ -14,7 +14,7 @@
 import java.math.MathContext;
 import java.math.RoundingMode;
 import java.nio.ByteBuffer;
-import java.nio.charset.Charset;
+import java.nio.charset.StandardCharsets;
 
 /**
  * Utility methods to facilitate serialization/deserialization
@@ -24,9 +24,6 @@
  */
 public class SerializationUtil {
 
-    /* TODO: Use StandardCharsets version in Java 8 */
-    private static final Charset utf8 = Charset.forName("UTF-8");
-
     public static final String EMPTY_STRING = new String();
 
     public static final byte[] EMPTY_BYTES = { };
@@ -208,7 +205,7 @@ private static String readStdUTF8String(ByteInputStream in)
         }
         final byte[] bytes = new byte[length];
         in.readFully(bytes);
-        return utf8.decode(ByteBuffer.wrap(bytes)).toString();
+        return StandardCharsets.UTF_8.decode(ByteBuffer.wrap(bytes)).toString();
     }
 
     /**
@@ -277,7 +274,7 @@ private static int writeStdUTF8String(DataOutput out, String value)
         if (value == null) {
             return writePackedInt(out, -1);
         }
-        final ByteBuffer buffer = utf8.encode(value);
+        final ByteBuffer buffer = StandardCharsets.UTF_8.encode(value);
         final int length = buffer.limit();
         int len = writePackedInt(out, length);
         if (length > 0) {

driver/src/main/java/oracle/nosql/driver/util/SimpleRateLimiter.java

@@ -98,6 +98,9 @@ private void enforceMinimumDuration() {
 
     @Override
     public double getLimitPerSecond() {
+        if (nanosPerUnit == 0) {
+            return 0.0;
+        }
         return 1_000_000_000.0 / (double)nanosPerUnit;
     }