[CARMEL-7521][CARMEL-4193][FOLLOWUP] Skip privilege check if authorization is disabled (apache#168)

Author: wakun

sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala

@@ -50,7 +50,7 @@ import org.apache.spark.sql.errors.QueryExecutionErrors.hiveTableWithAnsiInterva
 import org.apache.spark.sql.execution.datasources.{DataSource, DataSourceUtils, FileFormat, HadoopFsRelation, LogicalRelation}
 import org.apache.spark.sql.execution.datasources.delta.DeltaUtils
 import org.apache.spark.sql.execution.datasources.v2.FileDataSourceV2
-import org.apache.spark.sql.internal.{HiveSerDe, SessionState, SQLConf}
+import org.apache.spark.sql.internal.{HiveSerDe, SessionState, SQLConf, StaticSQLConf}
 import org.apache.spark.sql.types._
 import org.apache.spark.sql.util.PartitioningUtils
 import org.apache.spark.util.{SerializableConfiguration, ThreadUtils}
@@ -1160,12 +1160,14 @@ object DDLUtils extends Logging {
 
   def checkPrivilegeOfSpecifyTableLocation(
       location: Option[_], state: SessionState): Unit = {
-    val catalog = state.catalog
-    val user = catalog.getCurrentUser
-    val authorizer = state.authorizer
-    if (location.isDefined
-      && !authorizer.hasPrivilegeOfSpecifyTableLocation(user)) {
-      throw QueryCompilationErrors.createExternalTableUnsupported(catalog.getCurrentUser)
+    if (state.conf.getConf(StaticSQLConf.AUTHORIZATION_ENABLED)) {
+      val catalog = state.catalog
+      val user = catalog.getCurrentUser
+      val authorizer = state.authorizer
+      if (location.isDefined
+        && !authorizer.hasPrivilegeOfSpecifyTableLocation(user)) {
+        throw QueryCompilationErrors.createExternalTableUnsupported(catalog.getCurrentUser)
+      }
     }
   }