[SQL] address comments for to_date/trunc

[davies]

This PR address the comments in #7805

cc @rxin

[SparkQA]

Test build #39174 has finished for PR 7817 at commit ed1edc7.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • class MulticlassClassificationEvaluator (override val uid: String)
  • class NaiveBayes(JavaEstimator, HasFeaturesCol, HasLabelCol, HasPredictionCol):
  • class NaiveBayesModel(JavaModel):
  • class MulticlassClassificationEvaluator(JavaEvaluator, HasLabelCol, HasPredictionCol):

[rxin]

LGTM

[SparkQA]

Test build #39179 has finished for PR 7817 at commit 31e52ef.

• This patch fails to build.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • class MulticlassClassificationEvaluator (override val uid: String)
  • class NaiveBayes(JavaEstimator, HasFeaturesCol, HasLabelCol, HasPredictionCol):
  • class NaiveBayesModel(JavaModel):
  • class MulticlassClassificationEvaluator(JavaEvaluator, HasLabelCol, HasPredictionCol):

[SparkQA]

Test build #39185 has finished for PR 7817 at commit cb7f783.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #1257 has finished for PR 7817 at commit cb7f783.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • class MulticlassClassificationEvaluator (override val uid: String)
  • class NaiveBayes(JavaEstimator, HasFeaturesCol, HasLabelCol, HasPredictionCol):
  • class NaiveBayesModel(JavaModel):
  • class MulticlassClassificationEvaluator(JavaEvaluator, HasLabelCol, HasPredictionCol):

[SparkQA]

Test build #39229 has finished for PR 7817 at commit f729d5f.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.