Add timezone offset to string representation of TimestampValue

Author: gmfeinberg

CHANGELOG.md

@@ -25,7 +25,7 @@ principal with a delegation token for authorization and authentication:
 ### Fixed
 - Ensure that TableLimits is always null in TableResult on-premise.
 - Fixed a problem where SignatureProvider.getAuthorizationString may fail due to an IllegalStateException with error "Timer already cancelled".
-
+- Add timezone offset to the string representation of a TimestampValue to properly conform to ISO 8601 format.
 
 ### Changed
 - DefaultRetryHandler now uses incremental backoff mechanism (instead of fixed 1-second delay) and may be extended.

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

@@ -188,7 +188,7 @@ private static String formatString(Timestamp timestamp,
             return zdt.format(getDateTimeFormatter(fmt,
                                                    withZoneUTC,
                                                    optionalFracSecond,
-                                                   false));
+                                                   true));
         } catch (IllegalArgumentException iae) {
             throw new IllegalArgumentException("Failed to format the " +
                 "timestamp with pattern '" + fmt + "': " + iae.getMessage(),

driver/src/test/java/oracle/nosql/driver/ValueTest.java

@@ -452,7 +452,7 @@ public void testTimestamp() {
         }
 
         /* Test toJson(JsonOptions) */
-        text = "2017-07-15T15:18:59.123456789";
+        text = "2017-07-15T15:18:59.123456789Z";
         tsv = new TimestampValue(text);
 
         JsonOptions options = new JsonOptions().setTimestampAsString(true);
@@ -853,7 +853,7 @@ public void testSize() {
         assertSize(val, 12);
 
         val = new TimestampValue(tsText);
-        assertSize(val, 31);
+        assertSize(val, 32);
 
         val = BooleanValue.trueInstance();
         assertSize(val, 2);