drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From paul-rogers <...@git.apache.org>
Subject [GitHub] drill issue #1026: DRILL-5919: Add non-numeric support for JSON processing
Date Sat, 11 Nov 2017 01:47:47 GMT
Github user paul-rogers commented on the issue:

    https://github.com/apache/drill/pull/1026
  
    Further, is the extra option to `convertFromJSON` really needed? Can't we just accept
`NaN` and `Infinity` by default?
    
    Consider. If the option is off by default, users without `NaN` or `Infinity` data will
see no difference. But, users will this data will get an error and have to hunt down the option
to make their data work.
    
    If the option is on by default, users without `NaN` or `Infinity` data will see no difference.
But, users will this data will also have their queries work by default.
    
    So, seems no harm in making the `NaN` and `Infinity` support turned on by default.


---

Mime
View raw message