[GitHub] [drill] luocooong commented on pull request #2299: DRILL-7989: Use the UTC formatter in the JSON reader

[GitBox <gi...@apache.org>]

luocooong commented on pull request #2299:
URL: https://github.com/apache/drill/pull/2299#issuecomment-904520267


   @paul-rogers Thanks for the review. I think you said number is the `Canonical Format`? such as
   ```json
   {"$date":{"$numberLong":"1565546054692"}}
   ```
   I have tracked the mongo reader code, then found the mongo driver only uses the `Relaxed Format`.
   ```java
   // org.apache.drill.exec.store.mongo.MongoRecordReader
   String doc = cursor.next().toJson();
   jsonReader.setSource(doc.getBytes(Charsets.UTF_8));
   
   // com.mongodb.internal.connection.ByteBufBsonDocument
   public String toJson() {
       return toJson(JsonWriterSettings.builder().outputMode(JsonMode.RELAXED).build());
   }
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org