[SPARK-52369] Fix Session ID to be lowercased always

dongjoon-hyun · dongjoon-hyun · commit 40a4943b0976 · 2025-05-31T11:58:25.000-07:00
diff --git a/Sources/SparkConnect/SparkSession.swift b/Sources/SparkConnect/SparkSession.swift
@@ -44,9 +44,9 @@ public actor SparkSession {
     self.client = SparkConnectClient(remote: connection)
     // Since `Session ID` belongs to `SparkSession`, we handle this here.
     if connection.contains(regexSessionID) {
-      self.sessionID = connection.firstMatch(of: regexSessionID)!.1.uppercased()
+      self.sessionID = connection.firstMatch(of: regexSessionID)!.1.lowercased()
     } else {
-      self.sessionID = UUID().uuidString
+      self.sessionID = UUID().uuidString.lowercased()
     }
     self.conf = RuntimeConf(self.client)
   }

Original file line number	Diff line number	Diff line change
`@@ -44,9 +44,9 @@ public actor SparkSession {`
`44`	`44`	`self.client = SparkConnectClient(remote: connection)`
`45`	`45`	// Since `Session ID` belongs to `SparkSession`, we handle this here.
`46`	`46`	`if connection.contains(regexSessionID) {`
`47`		`- self.sessionID = connection.firstMatch(of: regexSessionID)!.1.uppercased()`
	`47`	`+ self.sessionID = connection.firstMatch(of: regexSessionID)!.1.lowercased()`
`48`	`48`	`} else {`
`49`		`- self.sessionID = UUID().uuidString`
	`49`	`+ self.sessionID = UUID().uuidString.lowercased()`
`50`	`50`	`}`
`51`	`51`	`self.conf = RuntimeConf(self.client)`
`52`	`52`	`}`