Addressed performance issue on Bulk Copy API with batch insert

Author: muskan124947

src/main/java/com/microsoft/sqlserver/jdbc/SQLServerBulkBatchInsertRecord.java

@@ -189,18 +189,7 @@ else if (dateTimeFormatter != null)
             case Types.LONGVARCHAR:
             case Types.NCHAR:
             case Types.NVARCHAR:
-            case Types.LONGNVARCHAR: {
-                /*
-                 * If string data comes in as a byte array through setString (and sendStringParametersAsUnicode = false)
-                 * through Bulk Copy for Batch Insert API, convert the byte array to a string.
-                 * If the data is already a string, return it as is.
-                 */
-                if (data instanceof byte[]) {
-                    return new String((byte[]) data, charset);
-                }
-                return data;
-            }
-
+            case Types.LONGNVARCHAR:
             case Types.DATE:
             case Types.CLOB:
             default: {

src/main/java/com/microsoft/sqlserver/jdbc/dtv.java

@@ -2063,7 +2063,8 @@ else if (type.isBinary()) {
             // then do the conversion now so that the decision to use a "short" or "long"
             // SSType (i.e. VARCHAR vs. TEXT/VARCHAR(max)) is based on the exact length of
             // the MBCS value (in bytes).
-            else if (null != collation && (JDBCType.CHAR == type || JDBCType.VARCHAR == type
+            // If useBulkCopyForBatchInsert is true, conversion to byte array is not done due to performance
+            else if ((con.getUseBulkCopyForBatchInsert() == false) && null != collation && (JDBCType.CHAR == type || JDBCType.VARCHAR == type
                     || JDBCType.LONGVARCHAR == type || JDBCType.CLOB == type)) {
                 byte[] nativeEncoding = null;