Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/22198#discussion_r212807880 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -144,7 +144,7 @@ class Analyzer( lazy val batches: Seq[Batch] = Seq( Batch("Hints", fixedPoint, - new ResolveHints.ResolveBroadcastHints(conf), + new ResolveHints.ResolveBroadcastHints(conf, catalog), --- End diff -- `catalog` is too big for this since we only use the current database name. ```scala - new ResolveHints.ResolveBroadcastHints(conf, catalog), + new ResolveHints.ResolveBroadcastHints(conf, catalog.getCurrentDatabase), ```
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org