nooneuse commented on code in PR #63143:
URL: https://github.com/apache/doris/pull/63143#discussion_r3279748080
##########
build.sh:
##########
@@ -662,6 +663,14 @@ FE_MODULES="$(
# Clean and build Backend
if [[ "${BUILD_BE}" -eq 1 ]]; then
+
+ echo "install datasketches-cpp to thirdparty path before build be"
+ update_submodule "contrib/datasketches-cpp" "datasketches-cpp"
"https://github.com/apache/datasketches-cpp/archive/refs/tags/5.2.0.tar.gz";
+ cd "${DORIS_HOME}/contrib/datasketches-cpp"
Review Comment:
> why maintain through `contrib` instead of `thirdparty`?
Hello, Thx for your comments @linrrzqqq ! This is mainly because I saw that
other third-party libraries based on git submodules, such as clucene and
apache-orc, are also maintained under the `contrib` folder, so I chose
`contrib`. Additionally, considering that datasketches-cpp is a header-only
library (hpp) that does not need to be built, only install required, so I did
not add it to `build thirdparty` either.
--
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]
