liuxiaocs7 commented on issue #6246: URL: https://github.com/apache/kyuubi/issues/6246#issuecomment-2044827071
Hi, @pan3793, I have browsed through what was done in https://github.com/apache/spark/pull/45154, which mainly accomplished the replacement of `Jakrata` to `Javax` namespaces in Spark-master, and indeed could be the main cause of this CI error. I tried to make some changes locally and found that there are currently some compilation issues mainly with the web related code under `Kyuubi Project Engine Spark SQL` module. In some implementations of the module, Kyuubi uses e.g. the `javax.servlet.http.HttpServletRequest` class, which leads to incompatibility when compiling with the latest Spark, which uses web-related class under `jakarta` namespace. But after modifying it to `jakarta`, other versions of Spark CI under Kyuubi reported the error again. It seems that `javax` and `jakarta` are difficult to deal with in the current scenario to pass all CI because Kyuubi uses classes such as `WebUIPage`/`UIUtils` from Spark WebUI, which also import classes in the corresponding namespace? -- 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]
