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.


---

Reply via email to