Do not support binary literal on Oracle

Signed-off-by: Xiaoguang Sun <[email protected]>

connector/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/v2/V2JDBCTest.scala

@@ -990,14 +990,19 @@ private[v2] trait V2JDBCTest extends SharedSparkSession with DockerIntegrationFu
   test("SPARK-50792 Format binary data as a binary literal in JDBC.") {
     val tableName = s"$catalogName.test_binary_literal"
     withTable(tableName) {
-      // Create a table with binary column
-      val binary = "X'123456'"
-
-      sql(s"CREATE TABLE $tableName (binary_col BINARY)")
-      sql(s"INSERT INTO $tableName VALUES ($binary)")
-
-      val select = s"SELECT * FROM $tableName WHERE binary_col = $binary"
-      assert(spark.sql(select).collect().length === 1, s"Binary literal test failed: $select")
+      // Unfornately, Oracle can only compare two BLOBs with a special function dbms_lob.compare
+      // The V2ExpressionSQLBuilder can not support rewriting the '=' operator.
+      // We don't test binary data on Oracle and do not support binary data on Oracle.
+      if (!this.isInstanceOf[OracleIntegrationSuite]) {
+        // Create a table with binary column
+        val binary = "X'123456'"
+
+        sql(s"CREATE TABLE $tableName (binary_col BINARY)")
+        sql(s"INSERT INTO $tableName VALUES ($binary)")
+
+        val select = s"SELECT * FROM $tableName WHERE binary_col = $binary"
+        assert(spark.sql(select).collect().length === 1, s"Binary literal test failed: $select")
+      }
     }
   }
 }

sql/core/src/main/scala/org/apache/spark/sql/jdbc/OracleDialect.scala

@@ -138,8 +138,6 @@ private case class OracleDialect() extends JdbcDialect with SQLConfHelper with N
     case timestampValue: Timestamp => "{ts '" + timestampValue + "'}"
     case dateValue: Date => "{d '" + dateValue + "'}"
     case arrayValue: Array[Any] => arrayValue.map(compileValue).mkString(", ")
-    case binaryValue: Array[Byte] =>
-      binaryValue.map("%02X".format(_)).mkString("HEXTORAW('", "", "')")
     case _ => value
   }