Revert "[SPARK-35321][SQL][3.0] Don't register Hive permanent functions when creating Hive client"

This reverts commit 5268a38.

Author: dongjoon-hyun

sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala

@@ -258,7 +258,7 @@ private[hive] class HiveClientImpl(
     if (clientLoader.cachedHive != null) {
       clientLoader.cachedHive.asInstanceOf[Hive]
     } else {
-      val c = shim.getHive(conf)
+      val c = Hive.get(conf)
       clientLoader.cachedHive = c
       c
     }
@@ -286,7 +286,7 @@ private[hive] class HiveClientImpl(
     // Set the thread local metastore client to the client associated with this HiveClientImpl.
     Hive.set(client)
     // Replace conf in the thread local Hive with current conf
-    shim.getHive(conf)
+    Hive.get(conf)
     // setCurrentSessionState will use the classLoader associated
     // with the HiveConf in `state` to override the context class loader of the current
     // thread.

sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala

@@ -173,8 +173,6 @@ private[client] sealed abstract class Shim {
 
   def getMSC(hive: Hive): IMetaStoreClient
 
-  def getHive(hiveConf: HiveConf): Hive
-
   protected def findMethod(klass: Class[_], name: String, args: Class[_]*): Method = {
     klass.getMethod(name, args: _*)
   }
@@ -197,8 +195,6 @@ private[client] class Shim_v0_12 extends Shim with Logging {
     getMSCMethod.invoke(hive).asInstanceOf[IMetaStoreClient]
   }
 
-  override def getHive(hiveConf: HiveConf): Hive = Hive.get(hiveConf)
-
   private lazy val startMethod =
     findStaticMethod(
       classOf[SessionState],
@@ -1237,13 +1233,6 @@ private[client] class Shim_v2_1 extends Shim_v2_0 {
   override def alterPartitions(hive: Hive, tableName: String, newParts: JList[Partition]): Unit = {
     alterPartitionsMethod.invoke(hive, tableName, newParts, environmentContextInAlterTable)
   }
-
-  // HIVE-10319 introduced a new HMS thrift API `get_all_functions` which is used by
-  // `Hive.get` since version 2.1.0, when it loads all Hive permanent functions during
-  // initialization. This breaks compatibility with HMS server of lower versions.
-  // To mitigate here we use `Hive.getWithFastCheck` instead which skips loading the permanent
-  // functions and therefore avoids calling `get_all_functions`.
-  override def getHive(hiveConf: HiveConf): Hive = Hive.getWithFastCheck(hiveConf, false)
 }
 
 private[client] class Shim_v2_2 extends Shim_v2_1