[KYUUBI #6708] spark-sql-engine's classes should not be visible in se…

…rver module

# 🔍 Description
## Issue References 🔗

The server module does not consume the `spark-sql-engine` classes, but it supposes that
`spark-sql-engine`'s jar existed on the target folder, so here we declare it as a dependency
to make sure that Maven always processes `spark-sql-engine` module before the server module.
IntelliJ IDEA 2024.1 fixed the IDEA-93855, thus the relocated classes inside the
`spark-sql-engine`'s shaded jar are visible in the server module in IDEA, for example,
`org.apache.kyuubi.shaded.spark.connect.proto.ExecutePlanRequest`, which silently breaks
the IDEA code analysis and jumping capabilities.

## Describe Your Solution 🔧

Changing the dependency type from `jar`(default value) to `pom` seems to be a workaround.

## Types of changes 🔖

- [x] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

## Test Plan 🧪

<img width="593" alt="image" src="https://github.com/user-attachments/assets/cc53709a-5f9d-4452-a24f-0c84e2342191">

#### Behavior Without This Pull Request ⚰️

<img width="1511" alt="image" src="https://github.com/user-attachments/assets/1c547c6d-a603-4c9d-92b4-8e2059b35fac">

#### Behavior With This Pull Request 🎉

<img width="1509" alt="image" src="https://github.com/user-attachments/assets/55f9e6e5-b9ea-4959-9142-ab7db21ab9b1">

---

# Checklist 📝

- [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes #6708 from pan3793/IDEA-93855.

Closes #6708

a73fd42 [Cheng Pan] fix
e4a8d36 [Cheng Pan] nit
8c58d2d [Cheng Pan] IDEA-93855

Authored-by: Cheng Pan <chengpan@apache.org>
Signed-off-by: Cheng Pan <chengpan@apache.org>
(cherry picked from commit 1d35cf2)
Signed-off-by: Cheng Pan <chengpan@apache.org>

kyuubi-server/pom.xml

@@ -317,10 +317,21 @@
  <scope>test</scope>
  </dependency>
 
+ <!--
+ The server module does not consume the `spark-sql-engine` classes, but it supposes that
+ `spark-sql-engine`'s jar existed on the target folder, so here we declare it as a dependency
+ to make sure that Maven always processes `spark-sql-engine` module before the server module.
+ IntelliJ IDEA 2024.1 fixed the IDEA-93855, thus the relocated classes inside the
+ `spark-sql-engine`'s shaded jar are visible in the server module in IDEA, for example,
+ `org.apache.kyuubi.shaded.spark.connect.proto.ExecutePlanRequest`, which silently breaks
+ the IDEA code analysis and jumping capabilities. It seems to be a workaround by changing
+ the dependency type from `jar`(default value) to `pom`.
+ -->
  <dependency>
  <groupId>org.apache.kyuubi</groupId>
  <artifactId>kyuubi-spark-sql-engine_${scala.binary.version}</artifactId>
  <version>${project.version}</version>
+ <type>pom</type>
  <scope>test</scope>
  <exclusions>
  <exclusion>