maomaodev commented on code in PR #7476:
URL: https://github.com/apache/kyuubi/pull/7476#discussion_r3301154375
##########
pom.xml:
##########
@@ -2069,6 +2069,7 @@
<module>extensions/spark/kyuubi-spark-connector-hive</module>
</modules>
<properties>
+ <maven.compiler.release>17</maven.compiler.release>
Review Comment:
The existing spark-4.0 profile is missing this property, while spark-4.1
already has it. Without it, scalac reports `Class java.lang.Record not found`
once a module references a JDK-17-only Spark 4.0 API (e.g. Aggregation's Record
type from SPARK-45919).
Inlining this one-line fix here to unblock CI, happy to split it out into a
follow-up PR if preferred.
<img width="1324" height="370" alt="Image"
src="https://github.com/user-attachments/assets/35bb2f4c-ae01-4d31-8bb6-1b23a3eea17b";
/>
--
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]
