You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/04/22 13:01:24 UTC

[GitHub] [spark] HyukjinKwon edited a comment on issue #24437: [SPARK-27512][SQL] Avoid to replace ', ' in CSV's decimal type inference for backward compatibility

HyukjinKwon edited a comment on issue #24437: [SPARK-27512][SQL] Avoid to replace ',' in CSV's decimal type inference for backward compatibility
URL: https://github.com/apache/spark/pull/24437#issuecomment-485409626
 
 
   Makes sense but I think that doesn't fully address the current issue - some people could still complain about changing this behaviour.
   
   IIUC, `prefersDecimal` is a switch to decide if we infer decimal or not. In this issue, looks like users still want the decimal inference but want to avoid one case containing `,`.
   
   Let's consider that option later when we drop this behaviour.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org