holdenk opened a new pull request, #59065:
URL: https://github.com/apache/spark/pull/59065

   ### What changes were proposed in this pull request?
   
   Backport the `extraServerConfs` hook in `SparkConnectServerTest` from master 
(added by SPARK-58094) so the SPARK-59087 backport's `SparkConnectAuthSuite` 
compiles on this branch.
   
   ### Why are the changes needed?
   
   The SPARK-59087 backport broke the connect test compile here: 
`SparkConnectAuthSuite` overrides `extraServerConfs`, which only exists on 
branches that have SPARK-58094 (master, branch-4.x, branch-4.3).
   
   ```
   [error] 
sql/connect/server/src/test/scala/org/apache/spark/sql/connect/service/SparkConnectAuthSuite.scala:56:26:
 method extraServerConfs overrides nothing
   ```
   
   ### Does this PR introduce _any_ user-facing change?
   
   No, test code only.
   
   ### How was this patch tested?
   
   `build/sbt -Phive connect/Test/compile` (red before, green now) and 
`build/sbt -Phive 'connect/testOnly *SparkConnectAuthSuite'` -- 3/3 pass.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Cursor
   
   Made with [Cursor](https://cursor.com)
   


-- 
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