Github user vladimirtkach commented on a diff in the pull request:
https://github.com/apache/drill/pull/1026#discussion_r150500059
--- Diff: exec/java-exec/src/main/resources/drill-module.conf ---
@@ -502,6 +502,8 @@ drill.exec.options: {
store.format: "parquet",
store.hive.optimize_scan_with_native_readers: false,
store.json.all_text_mode: false,
+ store.json.writer.non_numeric_numbers: false,
+ store.json.reader.non_numeric_numbers: false,
--- End diff --
No, we don't and we should test them. I think we have to create separate
jira for testing math functions, because code changes from this PR doesn't
affect logic of any math function, and while testing math function there will
be other issues not connected with with functionality (like BigInteger
constructor doesn't accept NaN, Infinity and others)
---
