qiaojialin commented on a change in pull request #1305:
URL: https://github.com/apache/incubator-iotdb/pull/1305#discussion_r435207961



##########
File path: 
server/src/main/java/org/apache/iotdb/db/utils/TypeInferenceUtils.java
##########
@@ -65,6 +64,9 @@ public static TSDataType getPredictedDataType(Object value, 
boolean inferType) {
         } else {
           return floatingStringInferType;
         }
+        // "NaN" is returned if the NaN Literal is given in Parser
+      } else if ("NaN".equals(strValue)) {
+        return TSDataType.DOUBLE;

Review comment:
       I prefer to add a configuration nan_string_infer_type=DOUBLE  by default 
and enable users to change this to TEXT.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to