Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add Instant/ZonedDateTime to JsDataHandler, too
Browse files Browse the repository at this point in the history
nbauernfeind committed Feb 28, 2023
1 parent ca14383 commit 09d150d
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -96,7 +96,7 @@ public void write(Object[] data, ParseContext context, JsConsumer<Node> addNode,
addNode.apply(new Node(data.length, nullCount));
}
},
DATE_TIME(Type.Int, "io.deephaven.time.DateTime", "datetime") {
DATE_TIME(Type.Int, "io.deephaven.time.DateTime", "datetime", "java.time.Instant", "java.time.ZonedDateTime") {
// Ensures that the 'T' separator character is in the date time
private String ensureSeparator(String s) {
if (s.charAt(SEPARATOR_INDEX) == ' ') {
@@ -466,7 +466,7 @@ private static void writeSimpleNumbers(Object[] data, JsConsumer<Node> addNode,
/**
* Helper to read some js value as a double, so it can be handled as some type narrower than a js number. Do not use
* this to handle wider types, check each possible type and fallback to this.
*
*
* @param data the data to turn into a js number
* @return null or a java double
*/

0 comments on commit 09d150d

Please sign in to comment.