Closed xleoken closed 3 months ago
cc @zhoujinsong
Hi @zhoujinsong for the suggestion. Patch is updated accordingly, can you please take another look?
Had tried following mvn commad in local environment.
mvn clean package
to package all modules except amoro-mixed-format-trino
.mvn clean package -Pbuild-mixed-format-trino -Ptoolchain
to package all modules.mvn clean package -Pbuild-mixed-format-trino -pl 'amoro-mixed-format/amoro-mixed-format-trino' -am
to package amoro-mixed-format-trino
only.Originally posted by @zhoujinsong in https://github.com/apache/amoro/issues/2853#issuecomment-2122373505
Why are the changes needed?
Close #2853.
Brief change log
It not friendly for users to use
mvn clean package -DskipTests -pl '!amoro-mixed-format/amoro-mixed-format-trino'
to build the amoro project. Add a newskip-trino-build
profile to improve the build command.How was this patch tested?
[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible
[ ] Add screenshots for manual tests if appropriate
[x] Run test locally before making a pull request
Documentation