weixi62961 commented on code in PR #5498:
URL: https://github.com/apache/kyuubi/pull/5498#discussion_r1369675848


##########
kyuubi-server/src/test/scala/org/apache/kyuubi/server/rest/client/BatchRestApiSuite.scala:
##########
@@ -238,4 +238,53 @@ class BatchRestApiSuite extends RestClientTestHelper with 
BatchTestHelper {
     assert(
       batchSession.conf.get(KyuubiReservedKeys.KYUUBI_CLIENT_VERSION_KEY) == 
Some(KYUUBI_VERSION))
   }
+
+  test("pyspark submit - basic batch rest client with existing resource file") 
{
+    val basicKyuubiRestClient: KyuubiRestClient =
+      KyuubiRestClient.builder(baseUri.toString)
+        .authHeaderMethod(KyuubiRestClient.AuthHeaderMethod.BASIC)
+        .username(ldapUser)
+        .password(ldapUserPasswd)
+        .socketTimeout(30000)
+        .build()
+    val batchRestApi: BatchRestApi = new BatchRestApi(basicKyuubiRestClient)
+
+    val requestObj = newBatchRequest(
+      batchType = PySparkJobPI.batchType,
+      resource = PySparkJobPI.resource.get,
+      className = null,

Review Comment:
   If we use the RestAPI client,  It need more effort to achieve this.
   please refer to the dto: 
   
https://github.com/apache/kyuubi/blob/ae29440453c35f4e541fb286effd6561d5234ba9/kyuubi-rest-client/src/main/java/org/apache/kyuubi/client/api/v1/dto/BatchRequest.java#L27C28-L27C28



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to