[GitHub] [spark] sadikovi commented on pull request #37327: [SPARK-39904][SQL] Rename inferDate to preferDate and add check for inferSchema = false
sadikovi commented on PR #37327: URL: https://github.com/apache/spark/pull/37327#issuecomment-1198896103 Yes, that was my thinking too. Okay, I will make a few changes to the PR. -- 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. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] [spark] sadikovi commented on pull request #37327: [SPARK-39904][SQL] Rename inferDate to preferDate and add check for inferSchema = false
sadikovi commented on PR #37327: URL: https://github.com/apache/spark/pull/37327#issuecomment-1198856750 Should we keep requirement that `inferDate = true` needs `inferSchema = true`? I think it is unclear right now. -- 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. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org