Copilot commented on code in PR #60715:
URL: https://github.com/apache/doris/pull/60715#discussion_r2797264219
##########
fe/fe-core/pom.xml:
##########
@@ -770,6 +770,12 @@ under the License.
<artifactId>mockito-inline</artifactId>
<scope>test</scope>
</dependency>
+ <!-- fastutil for memory-efficient primitive collections -->
+ <dependency>
+ <groupId>it.unimi.dsi</groupId>
+ <artifactId>fastutil-core</artifactId>
Review Comment:
There's a potential mismatch between the artifact declared in pom.xml
("fastutil-core") and the LICENSE file ("fastutil"). The standard fastutil
artifact is typically "it.unimi.dsi:fastutil", not "fastutil-core". The
"fastutil-core" is a modular variant introduced in fastutil 8.x. Please verify
that "fastutil-core" is the correct artifact to use, and if so, ensure the
LICENSE file reflects the actual artifact name consistently. If using the
standard "fastutil" artifact instead, update the pom.xml accordingly.
```suggestion
<artifactId>fastutil</artifactId>
```
--
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]
