[PECOBLR-620] Fixed byteBufferToString function and added unit tests (#908)

## Description
Fixed byteBufferToString function: The function assumed the byteBuffer
size to be a Long and resulted in buggy output for size 16. This
resulted in buggy value of sessionId and some logging statements.
Removed the extra logged statementId.

## Testing

- Tested locally

Related tickets:
[PECOBLR-620](https://databricks.atlassian.net/browse/PECOBLR-620)

[PECOBLR-620]:
https://databricks.atlassian.net/browse/PECOBLR-620?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ


`NO_CHANGELOG=true`

Author: msrathore-db

src/main/java/com/databricks/jdbc/common/DatabricksJdbcConstants.java

@@ -67,6 +67,7 @@ public final class DatabricksJdbcConstants {
   public static final String VOLUME_OPERATION_STATUS_COLUMN_NAME = "operation_status";
   public static final String VOLUME_OPERATION_STATUS_SUCCEEDED = "SUCCEEDED";
   public static final int VOLUME_OPERATION_MAX_RETRIES = 3;
+  public static final int UUID_LENGTH = 16;
 
   public static final String ARROW_METADATA_KEY = "Spark:DataType:SqlName";
   public static final Map<String, String> ALLOWED_SESSION_CONF_TO_DEFAULT_VALUES_MAP =

src/main/java/com/databricks/jdbc/common/util/DatabricksThriftUtil.java

@@ -6,6 +6,7 @@
 
 import com.databricks.jdbc.api.internal.IDatabricksSession;
 import com.databricks.jdbc.api.internal.IDatabricksStatementInternal;
+import com.databricks.jdbc.common.DatabricksJdbcConstants;
 import com.databricks.jdbc.dbclient.impl.common.StatementId;
 import com.databricks.jdbc.exception.DatabricksHttpException;
 import com.databricks.jdbc.exception.DatabricksSQLException;
@@ -56,8 +57,15 @@ public static TNamespace getNamespace(String catalog, String schema) {
 
   public static String byteBufferToString(ByteBuffer buffer) {
     ByteBuffer newBuffer = buffer.duplicate(); // This is to avoid a BufferUnderflowException
-    long sigBits = newBuffer.getLong();
-    return new UUID(sigBits, sigBits).toString();
+    // sessionId and statementID have guid which are 16 bytes long
+    if (newBuffer.remaining() >= DatabricksJdbcConstants.UUID_LENGTH) {
+      long mostSigBits = newBuffer.getLong();
+      long leastSigBits = newBuffer.getLong();
+      return new UUID(mostSigBits, leastSigBits).toString();
+    } else {
+      long sigBits = newBuffer.getLong();
+      return new UUID(sigBits, sigBits).toString();
+    }
   }
 
   public static ExternalLink createExternalLink(TSparkArrowResultLink chunkInfo, long chunkIndex) {
@@ -286,9 +294,7 @@ public static List<List<Object>> convertColumnarToRowBased(
   public static TOperationHandle getOperationHandle(StatementId statementId) {
     THandleIdentifier identifier = statementId.toOperationIdentifier();
     // This will help logging the statement-Id in readable format for debugging purposes
-    LOGGER.debug(
-        "getOperationHandle {%s} for statementId {%s}",
-        statementId, byteBufferToString(identifier.guid));
+    LOGGER.debug("getOperationHandle for statementId {%s}", byteBufferToString(identifier.guid));
     return new TOperationHandle()
         .setOperationId(identifier)
         .setOperationType(TOperationType.UNKNOWN);

src/test/java/com/databricks/jdbc/common/util/DatabricksThriftUtilTest.java

@@ -8,6 +8,7 @@
 
 import com.databricks.jdbc.api.internal.IDatabricksSession;
 import com.databricks.jdbc.api.internal.IDatabricksStatementInternal;
+import com.databricks.jdbc.common.DatabricksJdbcConstants;
 import com.databricks.jdbc.exception.DatabricksHttpException;
 import com.databricks.jdbc.exception.DatabricksSQLException;
 import com.databricks.jdbc.model.client.thrift.generated.*;
@@ -43,6 +44,20 @@ void testByteBufferToString() {
     assertEquals(expectedUUID, result);
   }
 
+  @Test
+  void testByteBufferToStringWithUuidLengthBytes() {
+    DatabricksThriftUtil helper = new DatabricksThriftUtil();
+    long mostSigBits = 987654321L;
+    long leastSigBits = 123456789L;
+    ByteBuffer buffer = ByteBuffer.allocate(DatabricksJdbcConstants.UUID_LENGTH);
+    buffer.putLong(mostSigBits);
+    buffer.putLong(leastSigBits);
+    buffer.flip();
+    String result = helper.byteBufferToString(buffer);
+    String expectedUUID = new UUID(mostSigBits, leastSigBits).toString();
+    assertEquals(expectedUUID, result);
+  }
+
   @Test
   void testVerifySuccessStatus() {
     assertDoesNotThrow(